Verbesserung der Leistung SP

Question

ich den Code unten haben ...

UPDATE Sales 
    SET MappingOrderID = CAST ([dbo].[fRemoveNonNumericCharacters] (ltrim(rtrim(ActualOrderNumber))) AS INT) 
    FROM Sales 
    where isnumeric(ActualOrderNumber) = 1 
    and ActualOrderNumber not like '%.%' 
    and len(ActualOrderNumber) < 10 
    and MappingOrderID is null 
    and ActualOrderNumber is not null 
    and ltrim(rtrim(ActualOrderNumber)) in (select ltrim(rtrim(OrderID)) from dbo.Orders) 

Die beteiligten Spalten sind:

aus dem Verkauf: MappingOrderId INT ActualOrderNumber NVARCHAR (10)

Von Aufträge: OrderId INT

Dies ist Legacy-Code, ich bin der DBA, ich weiß nicht, warum er/sie T verwendet so vielen Rand ... Das Skript dauert 9 Minuten zu laufen, und hat idealerweise häufig ausgeführt werden ...

ich dies auch im Ausführungsplan:

• CONVERT_IMPLICIT (varchar (12), [Datamart] . [DBO]. [Orders]. [OrderID],

  0)

• CONVERT_IMPLICIT (VARCHAR (20), [Datamart]. [DBO]. [Sales]. [ActualOrderNumber], 0)

• Typkonvertierung in Ausdruck (CONVERT_IMPLICIT (varchar (20), [DataMart]. [Dbo]. [Verkauf]. [ActualOrderNumber], 0)) kann "CardinalityEstimate" in der Abfrageplanauswahl beeinflussen, Typkonvertierung in Ausdruck (CONVERT_IMPLICIT (varchar (12), [Datamart]. [dbo]. [Aufträge]. [OrderID], 0)) kann "CardinalityEstimate" im Abfrageplan beeinflussen Wahl

Für das, was ich fühle ...

Es Updates MappingOrderId (int) durch ActualOrderNumber (varchar) Gießen ... Es nur aktualisieren, wenn ActualOrderNumber in Aufträge existiert ...

Ich versuche ein paar Modifikationen; aber der varchar ActualOrderNumber hat Werte wie 3545427103, und wenn ich versuche, alles zu entfernen, wie:

and len(ActualOrderNumber) < 10 

Es bricht ...

Mein Gesamtziel ist es, diesen Prozess zu machen die effizienteste möglich; stellt die Daten an der Quelle die einzig wahre Lösung dar?

Ich habe viele der Vorschläge implementiert und es ging bis zu einer Minute. Vielen Dank! Aber was ich nicht verstehe, ist dies:

Bevor mit dem hässlichen Code, den ich hatte:

Jetzt habe ich mehr liest ... und es dauert Sekunden zu laufen ... wie ist das möglich?

Answer 1

UPDATE Sales 
SET MappingOrderID = CAST (ltrim(rtrim(ActualOrderNumber)) AS INT) 
FROM Sales S 
inner join dbo.Orders O on ltrim(rtrim(O.OrderID))=ltrim(rtrim(s.ActualOrderNumber)) 
where isnumeric(ActualOrderNumber) = 1 
and ActualOrderNumber not like '%.%' 
and len(ActualOrderNumber) < 10 
and MappingOrderID is null 

Änderungen

1. Entfernte Funktion "fRemoveNonNumericCharacters"

• Wie Sie IsNumeric einchecken, wo Klausel, gibt es keine Notwendigkeit, diese Funktion zu nutzen.

2. Beigetreten Umsatz und Auftragseingang Tabelle

• wenn Sie Aufträge Tabelle in where-Klausel verwenden, wie Sie erwähnt haben, dann für jede Reihe von Verkaufstisch wird komplett Orders-Tabelle wiedergegeben werden.
• Um diesen Prozess zu reduzieren, habe ich hinzugefügt beizutreten.

3. entfernt ActualOrderNumeric nicht null ist

• Wie Sie überprüfen IsNumeric in where-Klausel, gibt es keine Notwendigkeit, diese Bedingung zu verwenden. Denn für Null gibt isnumeric 0 zurück.