Oracle
 sql >> Database >  >> RDS >> Oracle

Aggiunta di INNER JOIN a query con subquery con istruzione Inline Select

Senza alcuna istruzione DDL che definisca le tabelle è difficile da testare ma:

  1. 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 sia DD-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 esempio DATE '2016-01-01' .
  2. Hai il primo HAVING clausola prima del GROUP BY clausola - scambiali.
  3. 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 da WHERE clausola.
  4. Raggruppamento per A.TRADING_DWKEY||A.RM_SM_USER_CODE è inefficiente. Hai già A.TRADING_DWKEY come uno dei GROUP BY termini quindi non concatenare le stringhe includi solo l'altra colonna nel GROUP 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
);