SQL Optimierung V5R4 Teil II

[BenderD]

... sieht für mich auch nach einem Bug in der Query engine aus. Aufpassen muss man natürlich auch, dass man im Hauptselect nix doppelt kriegt (select distinct kunde oder so). Alternativ kann man da auch die Datei 24 mal dranjoinen, dann hat man die Umsätze pro Monat auch nebeneinander.

D*B

[Fuerchau]

Ggf. finden SQL die Bezüge nicht richtig da du die Table-ID's doppelt vergibst.

Tabelle P2 gibt es außerhalb des Subselects als auch innerhalb!
Mach dies mal eindeutig, wobei du nicht für jeden Subselect eine neue ID brauchst.

[KingofKning]

Hallo,
selbst wenn ich es umbaue dann bekomme ich einen SQL-Fehler.

select p1.adkto
, (select ist from planz where p1.adkto = knd and monat =
201301)
, (select ist from planz where p1.adkto = knd and monat =
201302)
, (select ist from planz where p1.adkto = knd and monat =
201303)
from soe01/adr01pf p1 where adkto = 11173

Vielleicht sollte ich doch nochmal schauen ob ich die UDF performanter bekomme.

GG

[B.Hauser]

Ich würde auch mal ganz lapidar sagen, dass 24 unabhängige SELECTs das Ganze auch nicht unbedingt performanter machen.
Dabei werden im Gegensatz zu der UDF (alle 24 Werte als String in einer Spalte) auch 24 einzelne Spalten generiert.
Ist das so gewollt?
Anstatt 24 einzelnen Sub-SELECTS würde ich es mit einer einzigen Common Table Expression oder einem einzigen Sub-Select in der FROM-Anweisung versuchen.

Select P1.adKto,
         Sum(Case When Monat = 201301 Then Coalesce(IST, 0) else 0),
         Sum(Case When Monat = 201302 Then Coalesce(IST, 0) else 0),
         Sum(Case When Monat = 201303 Then Coalesce(IST, 0) else 0),
         ...
From SOE01/ADR01PF p1 Left outer Join
    (Select Knd, Sum(Ist) Ist
        from Planz
        Group By Knd) x on p1.adkto = Knd
Where ....
Group by P1.ADKTO 
Order By P1.ADKTO;

oder

With x as (Select Knd, Sum(Ist)
          From Planz
          Group By Knd)
Select P1.adKto,
         Sum(Case When Monat = 201301 Then Coalesce(IST, 0) else 0),
         Sum(Case When Monat = 201302 Then Coalesce(IST, 0) else 0),
         Sum(Case When Monat = 201303 Then Coalesce(IST, 0) else 0),
         ...
From SOE01/ADR01PF p1 Left outer Join on P1.ADKTO = KND
Where ....
Group By P1.ADKTo
Order By P1.ADKTO;

Birgitta

[KingofKning]

Wenn du einen Cursor erstellst obwohl du nur einen Satz benötigst, musst du auch alle Sätze (auch wenn es nur 1 wird) in einer Schleife verarbeiten (die Syntax weiß ich im Moment nicht).Sonst bleibt der Cursor offen und kann im Wiederholungsfall nicht noch mal verwendet werden.Der Return muss also hinter das End!Alternativ geht auch ein "select ... into ....", dann gibt's keinen Cursor.

Das End ist da schon richtig. Das Return steht hinter dem "end for"Irgendwie verhuddelt mein Firefox auf der Arbeit die Text und nimmt alle Leerzeichen etc. raus. Zuhause der Firefox läßt alles so wie es ist. Ich bin noch nicht hinter dieses nervige Geheimnis gekommen. Evt versuche ich es mal mit dem IE obwohl ich den nicht mag..GG