Discussion :

[LandGreen]

Bonjour tout le monde,

ma requête ci dessous étant très lente elle prend 13 minutes pour afficher le résultat,
ma question : y a t'il un moyen pour l'optimiser afin d'obtenir un résultat dans un temps un peu réduit ?

NB : il s'agit de trouver la différence entre SUM_TVA_REAL et TVA_COLLECT puisque le

[escartefigue]

Bonjour,

Déjà commencez par remplacer les jointures NATURAL JOIN extrêmement dangereuses par des jointure explicites, ce sera plus clair et surtout plus stable en terme de résultat !

Vous pouvez aussi tenter de remplacer les sous requêtes IN par un test d'existence avec EXISTS.

Ensuite, existe -t- il des index pour les critères de filtrage et de jointure ?

[Waldar]

Optimiser une requête sans trace, ni plan d'exécution, ni structure des tables / index en jeu c'est compliqué.
Il faut savoir où passe le temps.

Ensuite, je rejoins escartefigue, si vous utilisez NATURAL JOIN, vous implémentez un bogue.
Il s'agit d'une mauvaise fonctionnalité inventée dans je ne sais plus quelle version de la norme ANSI qui est stupide.

Arrêtez tout de suite et prenez 20 secondes de plus pour écrire explicitement vos critères de jointures.

Enfin, je n'ai pas tout regardé mais le début de votre code - vos curseurs A et B peuvent être remplacés par une seule requête.
D'ailleurs quelle étrange idée d'appeler ça curseur... même si ce n'est fondamentalement pas faux, c'est confusant.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
with cte_base as
(
select BRA_CODE, TRA_DATE, TRA_SEQ1
  from TRANSACT
 where EXPL_CODE = 368
   and DOC_ALP   = '31'
   and TRA_DATE >= date '2017-02-09'
)
  select trs.BRA_CODE, trs.TRA_DATE, trs.TRA_SEQ1
       , round(sum(case trs.EXPL_CODE when 2049 then 1 else 0    end * trs.TRA_AMT), 2) as tva_collect
       , round(sum(case trs.EXPL_CODE when 2049 then 0 else 0.19 end * trs.TRA_AMT), 2) as tva_reelle
    from TRANSACT trs
   where exists (select null
                   from cte_base cbs
                  where cbs.BRA_CODE = trs.BRA_CODE
                    and cbs.TRA_DATE = trs.TRA_DATE
                    and cbs.TRA_SEQ1 = trs.TRA_SEQ1)
   where trs.EXPL_CODE  in (2049, 2124, 2127)
     and trs.DEB_CRE_IND = 2
group by trs.BRA_CODE, trs.TRA_DATE, trs.TRA_SEQ1
  having round(sum(case trs.EXPL_CODE when 2049 then 1 else 0    end * trs.TRA_AMT), 2)
      <> round(sum(case trs.EXPL_CODE when 2049 then 0 else 0.19 end * trs.TRA_AMT), 2);

[LandGreen]

Je suis débutant, j'ai suivis "ojo77"