Senza alcuna istruzione DDL che definisca le tabelle è difficile da testare ma:
- Stai confrontando le date utilizzando stringhe:Oracle sta probabilmente eseguendo una conversione implicita in una data utilizzando il valore di
NLS_DATE_FORMAT
parametro di sessione (che si spera siaDD-MON-YY
) ma se questo dovesse mai cambiare, interromperà la query (e la query non sarà cambiata, quindi sarà difficile eseguire il debug). Smetti di usare le stringhe come date e usa i valori letterali della data, ad esempioDATE '2016-01-01'
. - Hai il primo
HAVING
clausola prima delGROUP BY
clausola - scambiali. - Stai anche combinando un join con virgola legacy e un join ANSI:sostituisci il join con virgola con un
INNER JOIN
e sposta la condizione di unione fuori daWHERE
clausola. - Raggruppamento per
A.TRADING_DWKEY||A.RM_SM_USER_CODE
è inefficiente. Hai giàA.TRADING_DWKEY
come uno deiGROUP BY
termini quindi non concatenare le stringhe includi solo l'altra colonna nelGROUP BY
clausola.
Qualcosa come:
Select TRADING_DWKEY,
RM_SM_NAME,
FIRST_TRADE,
'New Footprint' Status,
Null Q4Vol,
CQVol
From (
Select A.TRADING_DWKEY,
q.RM_SM_NAME,
B.FIRST_TRADE,
SUM(A.RISK_AMOUNT_ADJ)/1000000 CQVol
from FACT_TRADE_ROLLUP A
INNER JOIN
( SELECT TRADING_DWKEY,
MIN(FIRST_TRADE_DATE) FIRST_TRADE
FROM ACCOUNT_FIRST_LAST_TRADE_DATES
GROUP BY TRADING_DWKEY
HAVING MIN(FIRST_TRADE_DATE)>= trunc(sysdate-1,'Y')
) B
ON ( A.TRADING_DWKEY=B.TRADING_DWKEY )
INNER JOIN REF_RM_SM_REGION q
ON ( q.RM_SM_USER_CODE = A.RM_SM_USER_CODE )
WHERE A.DATE_KEY >= DATE '2016-01-01'
AND ( ( A.PRODUCT_DWKEY IN('RT')
AND A.Account_Type='Customer'
)
OR A.PRODUCT_DWKEY IN('OB','BS','MM')
)
AND A.Role = 'SM'
GROUP BY A.RM_SM_USER_CODE,
A.TRADING_DWKEY,
q.RM_SM_NAME,
B.FIRST_TRADE
HAVING SUM(A.RISK_AMOUNT_ADJ)>=20000000
);