Performance

Re: Performance

Also ich weiß nicht, wie viele Einträge insgesamt rausgelesen werden, aber deine Variante im Loop sieht wirklich nicht ganz so gut aus. Das liegt aber hauptsächlich daran, dass du in jedem Durchlauf eine Tabelle kopierst und dann mit DELETE nochmal über die (wahrscheinlich) unsortierte Kopie drüber musst.

Das vereinfachte Problem, das du schilderst, sieht für mich sehr nach JOIN aus. Geht es denn damit nicht in einem Select? Man sollte hier beachten, dass der Transport überflüssiger Daten von DB nach Applikationsserver die Performance auch nach oben treiben kann.

Außerdem könntest du dir mal LOOP AT ... GROUP BY anschauen. Das ist auch eine neuere Funktion für Schleifen und gibt dir die Möglichkeit, die große, äußere Tabelle in kleinen Gruppen zu durchlaufen.

Kleine Anmerkung noch: es ist immer ratsam eine IF ... IS NOT INITIAL-Abfrage um Selects mit FOR ALL ENTRIES zu machen, da bei leerer Tabelle einfach alles geladen wird. Nicht gut ^^

Re: Performance

schick hat geschrieben:
Spontan kommt mir sowas als Option in den Sinn, ich bin mir aber nicht sicher, ob das wirklich performanter ist... (sonderlich "elegant" ist es ohnehin nicht):

Code: Alles auswählen.

SELECT * 
FROM vbak 
INTO TABLE lt_vbak.

SELECT *
FROM vbap
INTO TABLE lt_vbap
FOR ALL ENTRIES IN lt_vbak.

LOOP AT lt_vbak INTO ls_vbak.

lt_vbap_neu = lt_vbap.
DELETE lt_vbap_neu WHERE vbeln NE lt_vbak-vbeln.

...

CALL FUNCTION 'X'
EXPORTING
TABLES 
position = lt_vbap_neu
EXCEPTIONS.

...

ENDLOOP.

Ich freue mich über Anregungen!


Randbemerkung:
Sowohl "SELECT *" als auch die "tollen" Bezeichner ala "lt_vbak" sind nur hier als Platzhalter, im richtigen Programm ist das anders gelöst...

Edit:
Releasestand ist übrigens 7.4

Es ist natürlich schwer eine 100%tige Aussage zu treffen, was besser ist ohne das Programm zu kennen.

Beispiel : Werden keine Werte tatsächlich von VBAK gebraucht, wäre ein Subquery beim Select auf die VBAP schon mal besser.

Wenn VBAK explizit sortiert wird im Programm, würde ich schon mal gleich in einen Tabelle mit einem sorted-Tabellentyp selektieren. Aber auch diese Frage ist bei FOR ALL ENTRIES nur relevant, wenn auf den Feldern, auf die in VBAP selektiert wird ein Index liegt.

Ob der FuBA eine Gruppe aus Group by aktzeptiert kann ich gerade mangels aktuellem System nicht sagen.

Das obige Beispiel mit dem Delete ist jedenfalls denkbar schlecht. ( 1. Speicherverbrauch ) : Du kopierst erst n-Zeilen in eine interne Tabelle ( und zwar jedes mal die "riesige" ursprüngliche interne Tabelle um ( 2. Performance ) : n-mal nicht benötigte Zeilen zu löschen.

Ein Zahlenbeispiel : Gegeben seien 1 Mio Kundenaufträge mit jeweils 1 Position : VBAK hat 1 Mio. Zeilen. VBAP liefert dazu exakt auch 1 Mio. Zeilen. Das Programm braucht also Speicher für 1 Mio VBAP Zeilen. Bei dem ursprünglichen Coding würde jedoch nur Speicher für exakt 1 VBAP-Zeile gebraucht. 2. Du löscht 999.999 mal 999.999 unbenötigte VBAP Zeilen.

Gruß,

GTO