Discussion :

[Oluha]

Bonjour à tous

J'aimerai savoir s'il existe des "règles" pour optimiser l'execution de procédures stockées.
En effet, j'ai crée une procédure stockée pour mon programme qui permet de corriger automatiquement les CP/VILLE faux à l'aide d'un fichier de la Poste, et je la trouve vraiment trop longue à s'executer.
Actuellement je fais des tests sur une base de 1661 enregistrements faux qui fait appelle à la table des CP/VILLE de 60554 enregistrements. La procédure stockée s'execute en un peu plus de 15min ce qui est vraiment trop long !

J'ai essayé de faire pas mal de modifications mais je n'arrive pas à faire baisser ce temps d'execution. Je veux bien mettre le code de ma procédure mais elle est assez longue...

Existe-t-il des astuces pour diminuer le temps d'execution d'une procédure stockée ?

Merci pour vos réponses.

[mohamed]

S'il y' a bcp d'indexes sur ta (tes) tables, les requêtes/proc stocks d'insertion/update seront bcp plus lents (contrairement aux requêtes select).
Ca peut etre du à cela!
sinon tu pourrais utiliser le profiler et exécuter la/les requêtes pour voir kels sont les lignes de requêtes qui prennent le plus de temps!!

[Fabien Celaia]

A quoi ressemble votre SP ?
A quoi ressemble le plan d'exécution ?

C'est peut-être au niveau de vos jointures que ça coince...

[Oluha]

j'ai en effet pas mal d'index sur mes tables car je ne pensais pas que ca ralentissait les updates. Pour ma table que je corrige, je peux créer ces index après la procédure, par contre sur la table de CP/VILLE pas moyen car cette table sert aussi à pleins d'autres programmes qui font des select dessus.

Sinon voici un schéma de ma procédure :

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
CREATE Procedure CorrCP
AS
SET nocount OFF
SET LOCK_TIMEOUT -1
 
DECLARE ...
 
DECLARE curs SCROLL CURSOR FOR
select id,cp,ville,anomalie,left(cp,2) as dpt ...
from tabtrav
where anomalie = 'CP' order by id
OPEN curs
FETCH FIRST FROM curs INTO @I, @CP, @VILLE ...
WHILE @@fetch_status = 0
BEGIN
  IF @ANOMALIE = 'CP'
  BEGIN
    SET ....
    IF (@CP <> '' AND @CP <> null)
    BEGIN
       SET @NBSOL = (SELECT count(*) FROM loccp WHERE ...
       ...
       IF @NBSOL = 1
       BEGIN
         SET @CORRCP = (SELECT CP FROM loccp WHERE ...)
         SET @CORRVILLE = ....
         SET @ANOMALIE = 'CC'
         GOTO sortie
       END
       -- 15 autres conditions ayant la même structure :
       IF @ANOMALIE = 'CP'
       BEGIN
         SET @NBSOL = ...
       END
       .
       .
       .
    END
    sortie:
    -- mise à jour de toutes les lignes ayant la même erreur cp/ville
    SET ANOMALIE = @ANOMALIE, CORRCP = @CORRCP, CORRVILLE = @CORRVILLE WHERE CORRCP = @CPORIG AND CORRVILLE = @VILLEORIG
  END
  FETCH NEXT FROM curs INTO ...
END
CLOSE curs
DEALLOCATE CURS

[LooserBoy]

Quelques bonnes astuces pour optimiser ses requètes SQL:
http://sqlpro.developpez.com/cours/optimiser/#L9

[Oluha]

merci pour ce lien très instructif que je n'avais pas vu sur le site. J'avais déjà essayé d'utiliser le plan d'execution sous l'analyseur mais il me trouvait que des 0% et des 2%.

Je viens de résoudre mon problème : j'appelais pas mal de fois une fonction que j'ai créée. Maintenant je l'appelle une seule fois dans le select de mon curseur et ca va nettement plus vite ! Je passe d'environ 15min à 2min !