Discussion :

[Deedoo2000]

Bonjour,

Je rencontre de gros problèmes de performances avec une de mes procs.
Voilà résumé le problème :
Le gros de cette proc est d'insérer dans une table via un curseur.
Comme vous pouvez le voir plus bas, une jointure a été supprimée (AND ACTNAIRE_NOMI.CD_VALEUR = @CD_VALEUR). Cette jointure apparaissait à plusieurs endroits de la proc, et depuis les performances ont chuté.
Il n'y avait pas d'index portant sur cette colonne mais pourtant les perf étaient bonnes. Mais depuis qu'on a supprimé cette jointure, sans toucher aux indexs, les perfs ont chuté.

Pouvez-vous m'aider ? Merci

Pour les courageux je peux envoyer le script complet de la proc

Exemple d'un bout de la proc :

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
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
 
DECLARE cur_nomi CURSOR
FOR
SELECT  ID_NOMINATIF,
        ID_TYPE_PROP,
        ID_CIVIL,
        NOM,
        PRENOM,
        BATIMENT,
        RUE,
        COMPLEMENT,
        CODE_POSTAL,
        VILLE,
        PAYS,
        NB_ACT_VS,
        NB_ACT_VP,
        REF_SOCIETE,
        REF_SALARIE,
        CD_REG_COU,
        CD_SS_REG_COU,
        CD_VALEUR,
        ADRE_PAYS_IDEN,
        IND_BLOCAGE,
        LANG
 
FROM    NOMINATIF_TRANSIT
        WHERE ID_ASS_GEN  = @ID_ASS_GEN
          AND CD_COMPUTER = @CD_COMPUTER
          AND CD_USER     = @CD_USER
 
 
open cur_nomi
fetch cur_nomi into @ID_NOMINATIF,
                    @ID_TYPE_PROP,
                    @ID_CIVIL,
                    @NOM,
                    @PRENOM,
                    @BATIMENT,
                    @RUE,
                    @COMPLEMENT,
                    @CODE_POSTAL,
                    @VILLE,
                    @PAYS,
                    @NB_ACT_VS,
                    @NB_ACT_VP,
                    @REF_SOCIETE,
                    @REF_SALARIE,
                    @CD_REG_COU,
                    @CD_SS_REG_COU,
                    @CD_VALEUR,
                    @ADRE_PAYS_IDEN,
                    @IND_BLOCAGE,
                    @LANG
 
 
/* Boucle sur les lignes */
WHILE @@sqlstatus = 0
BEGIN
 
   /* Recherche dans la base des infos existantes concernant cet actionnaire */
   SELECT   @ID_ACTNAIRE      = ACTNAIRE.ID_ACTNAIRE,
            @ID_ACTNAIRE_NOMI = ACTNAIRE_NOMI.ID_ACTNAIRE,
            @NB_ACT_VS_OLD    = ACTNAIRE_NOMI.NB_ACT_VS,
            @NB_ACT_VP_OLD    = ACTNAIRE_NOMI.NB_ACT_VP
 
   FROM   ACTNAIRE, ACTNAIRE_NOMI
 
   WHERE ACTNAIRE.ID_ASS_GEN        =  @ID_ASS_GEN
   AND   ACTNAIRE.ID_NOMINATIF      =  @ID_NOMINATIF
   AND   ACTNAIRE_NOMI.ID_ASS_GEN   =  @ID_ASS_GEN
   AND   ACTNAIRE_NOMI.ID_ACTNAIRE  =  ACTNAIRE.ID_ACTNAIRE
   AND   ACTNAIRE_NOMI.ID_TYPE_PROP =  @ID_TYPE_PROP
   -- Supression du contrôle sur le code valeur
   -- AND   ACTNAIRE_NOMI.CD_VALEUR    =  @CD_VALEUR

[mpeppler]

Il faudrait voir le SHOWPLAN ainsi que la définition des indexes sur les deux tables pour voir ce qui ce passe.

Michael

[Deedoo2000]

J'ai justement un problème avec le showplan.

J'essaie de l'obtenir dans SQL Advantage et j'ai le message suivant quand j'execute le script de ma proc :

DECLARE CURSOR must be the only statement in a query batch.

Et du coup je n'arrive pas à avoir mon plan... Que faire ?

[mpeppler]

Soit il faut exécuter la proc elle-même avec le SHOWPLAN, soit il faut splitter le code en deux partie et executer le "DECLARE ... CURSOR" séparément. Dans isql cela fait:

Code :

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

1
2
3
4
5
6
7
8
 
set showplan on
set statistics io on
go
declare ... cursor for select ...
go
... reste du code
go

Michael

[Deedoo2000]

Soit il faut exécuter la proc elle-même avec le SHOWPLAN, soit il faut splitter le code en deux partie et executer le "DECLARE ... CURSOR" séparément. Dans isql cela fait:

Code :

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

1
2
3
4
5
6
7
8
 
set showplan on
set statistics io on
go
declare ... cursor for select ...
go
... reste du code
go

Michael

Ca ne marche toujours pas
Voila à quoi ressemble mon script :


set showplan on
set statistics io on
go
...code...
go
declare cursor for select
go
reste du code
go

et j'obtiens toujours ceci :

QUERY PLAN FOR STATEMENT 1 (at line 1).




STEP 1
The type of query is SET OPTION ON.


Server Message: Number 3630, Severity 10
Server 'TIT1_PAR_TST_SQL', Line 3:
Total estimated I/O cost for statement 1 (at line 1): 0.




QUERY PLAN FOR STATEMENT 2 (at line 2).




STEP 1
The type of query is SET STATISTICS ON.


Server Message: Number 3630, Severity 10
Server 'TIT1_PAR_TST_SQL', Line 3:
Total estimated I/O cost for statement 2 (at line 2): 0.




QUERY PLAN FOR STATEMENT 3 (at line 3).




STEP 1
The type of query is SET OPTION ON.


Server Message: Number 3630, Severity 10
Server 'TIT1_PAR_TST_SQL', Line 3:
Total estimated I/O cost for statement 3 (at line 3): 0.


Server Message: Number 7344, Severity 15
Server 'TIT1_PAR_TST_SQL', Line 122:
DECLARE CURSOR must be the only statement in a query batch.

et ensuite plein de messages d'erreurs pour des variables non déclarées mais qui l'ont pourtant été dans la première partie du code...

As-tu une idée ?

[mpeppler]

Est-ce que SQLAdvantage connait le "go" comme isql?

Est-ce qu'il y a bien un "go" après le declare ... cursor, et PAS de declare ... cursor dans le batch suivant?

Michael