Query taking lot of time (Tunning Query.... Help)

Question

SELECT d.loc_channel, A.DEPOT_CODE, F.DEPOT_NAME,E.EMP_CODE,E.EMP_NAME, A.CUST_CODE, B.CUST_NAME, A.INSTRUMENT_NO,to_char(A.INSTRUMENT_DT,'dd/mm/yyyy')as INSTRUMENT_DT ,nvl(A.amount,0)as amount, G.REMARKS indorcvdfrombank,G.DOC_NO,to_char(G.DOC_DT,'dd/mm/yyyy')as doc_dt,g.doc_dt docdt,nvl(G.DOC_AMOUNT,0)as doc_amount,decode(nvl(c.BDSNo,'N') ,'N','No','Yes') PAYRCVD,c.BDSNO,to_char(c.BDSDT,'dd/mm/yyyy')as BDSDT,nvl(c.amount,0)as bdsamount FROM T_PAYRECEIPT A, M_CUSTOMER B,T_PAYRECEIPT C,M_LOCATION D, M_EMPLOYEE E, M_DEPOT F, T_CRD_DBT G WHERE A.CUST_CODE = B.CUST_CODE AND B.SE_LOCCODE = D.LOCATION_CODE AND D.EMP_CODE = E.EMP_CODE AND A.DEPOT_CODE = F.DEPOT_CODE AND A.REF_DOCNO = G.DOC_NO AND A.DEPOT_CODE = G.DEPOT_CODE and nvl(A.status,' ') = 'B' and a.voucher_no = c.ref_instrumentno(+) and trunc(a.voucher_dt) between to_date('19/04/2005','dd/mm/yyyy') and to_date('19/05/2005','dd/mm/yyyy') AND A.Cust_Code exists (select A.cust_code from M_Customer A) AND A.Depot_code exists (select depot_Code from M_DEPOT where DEPOT_CATEGORY in ('D','L') and status = 'A' ) order by a.cust_code,g.doc_no,docdt

Answer 1

While I agree with all of the other John's comments, as a first crack, I would re-write the statement this way:
SELECT d.loc_channel, a.depot_code, f.depot_name,e.emp_code,e.emp_name,
       a.cust_code, b.cust_name, a.instrument_no,
       TO_CHAR(a.instrument_dt,'dd/mm/yyyy') instrument_dt,
       NVL(a.amount,0) amount, g.remarks indorcvdfrombank,
       g.doc_no,TO_CHAR(g.doc_dt,'dd/mm/yyyy') doc_dt, g.doc_dt docdt,
       NVL(g.doc_amount,0) doc_amount,
       DECODE(NVL(c.bdsno,'N'), 'N', 'No', 'Yes') payrcvd,
       c.bdsno,TO_CHAR(c.bdsdt,'dd/mm/yyyy') bdsdt,
       NVL(c.amount,0) bdsamount
FROM t_payreceipt a, m_customer b,t_payreceipt c,m_location d,
     m_employee e, m_depot f, t_crd_dbt g
WHERE a.cust_code = b.cust_code and
      b.se_loccode = d.location_code and
      d.emp_code = e.emp_code and
      a.depot_code = f.depot_code and
      a.ref_docno = g.doc_no and
      a.depot_code = g.depot_code and
      a.status = 'b' and
      a.voucher_no = c.ref_instrumentno(+) and
      TRUNC(a.voucher_dt) BETWEEN TO_DATE('19/04/2005','dd/mm/yyyy') AND
                                  TO_DATE('19/05/2005','dd/mm/yyyy') and
      f.depot_category IN ('d','l') and
      f.status = 'a'
ORDER BY a.cust_code,g.doc_no,docdt The predicate:
a.cust_code EXISTS (SELECT a.cust_code FROM m_customer a)can be removed because the join condition:
a.cust_code = b.cust_codedoes exactly the same thing.
The predicate:
a.depot_code EXISTS (SELECT depot_code
                     FROM m_depot
                     WHERE depot_category IN ('d','l') and
                           status = 'a' ) can be removed because the join between:
a.depot_code = f.depot_codewill take care of it if you add the conditions into the main query lilke:
f.depot_category IN ('d','l') and
f.status = 'a'The predicate:
NVL(a.status,' ') = 'b'can be changed to a simple equality since NULL is not equal to anything. Removing the function may allow the optimizer to choose a better access path.
I would try to change the predicate:
TRUNC(a.voucher_dt) BETWEEN TO_DATE('19/04/2005','dd/mm/yyyy') AND
                            TO_DATE('19/05/2005','dd/mm/yyyy')to get rid of the TRUNC on voucher_dt. If there really are times in voucher_dt, I would probably use something like:
a.voucher_dt BETWEEN TO_DATE('19/04/2005','dd/mm/yyyy') AND
                     TO_DATE('19/05/2005 23:59:59','dd/mm/yyyy hh24:mi:ss')If there are no times in voucher_dt then just drop the TRUNC.
TTFN
John