Discussion :

[castorcharly]

Bonjour,

Je rencontre un problème dans une PS, que je n'arrive à reproduire en test.

Dans cette PS je déclare un cursor selon

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
DECLARE CurEre CURSOR FOR SELECT....
If champ = xx begin   
BEGINTRANSACTION
update ....
update ....
COMMIT TRANSACTION
end
FETCH NEXT FROM CurEre
....
CLOSE CurEre

Le but étant de parcourir une table et selon le contenu de "champ" je fais deux updates dans d'autres tables en démarrant une transaction qui est commité juste après les updates.

Dans certain cas, que je n'ai pu isoler, j'ai un message me signifiant que:
La transaction (id N°xx) a été bloquée sur les ressources verrou par un autre processus et a été chosie comme victime.

Je comprends bien le sens de ce message, mais je voudrais savoir si c'est juste pendant le BeginTrans...Commit que ce message est généré ou bien le fait d'être sous une déclaration CURSOR ?
Ou autrement, le fait de déclarer un CURSOR pose t il des verrous sur la table concernée ?

Merci pour votre aide,

[aieeeuuuuu]

Bonjour

Pourquoi utilisez vous un curseur pour faire ça. Vous n'avez pas mis les requêtes, mais il me semble que vous pourriez faire quelque chose du genre :

Code :

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

1
2
3
4
5
6
7
8
 
BEGIN TRAN
UPDATE ...
WHERE colonne = 'xx'
 
UPDATE ...
WHERE colonne = 'xx'
COMMIT TRAN

[castorcharly]

Bonjour

Pourquoi utilisez vous un curseur pour faire ça. Vous n'avez pas mis les requêtes, mais il me semble que vous pourriez faire quelque chose du genre :

Code :

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

1
2
3
4
5
6
7
8
 
BEGIN TRAN
UPDATE ...
WHERE colonne = 'xx'
 
UPDATE ...
WHERE colonne = 'xx'
COMMIT TRAN

Bonjour,

merci pour votre réponse,

Effectivement, je n'ai pas mis la totalité du code pour simplifier la question.

En fait l'objectif est de parcourrir un ensemble de résultat suite à une requête et en fonction de la valeur de certain champ de cette requete effectuer ou non un ou des updates.


Sous delphi je ferai un truc comme ça (en "code" de principe):
Sur un select qui renvoie n lignes

Code :

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

1
2
3
4
5
6
7
8
9
10
 
while not eof do begin
        if fieldbyname('champ').asinteger:= 1 then begin
           startTransaction;
           update table1 set tbl1champ = 0 where tbl1ChampId = :id;
           update table2 set tbl2champ = 999 where tbl2ChampId = :id;
           commit;
        end;
      next;
end;

Mais je pense que c'est plus efficace de tout faire dans la PS.

Ma question était juste, lorsque l'on déclare un curseur, y a t il un verrou de posé ?


Maintenant, sur l'opportunité ou non d'utiliser un cursor, là je ne sais pas.

[SQLpro]

...

Ma question était juste, lorsque l'on déclare un curseur, y a t il un verrou de posé ?

Tout dépend de la façon dont vous avez déclarer le curseur.

Si vous avez juste fait :

Code :

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

DECLARE CurEre CURSOR FOR SELECT....

Vous avez créé un curseur dynamique, scrollable, misajourable....
Si vous ne faites pas de mise à jour dans les lignes des tables du curseur, si vous n'avez pas beoin de revenir en arrière ou de vous positionner à la ligne 36541 et si vous ne voulez pas que le curseur reflète les valeurs modifiées par d'autres utilisateurs, alors il faut utiliser différentes options, parmi lesquelles :

Code :

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

LOCAL  FORWARD_ONLY STATIC READ_ONLY

De ce fait vous n'aurez pas de verrou.

Cependant il est idiot d'utiliser un curseur qui ne peut pas être optimisé et dans votre cas génère des verrous mortels, au lieu d'une requête qui elle peut l'être.

A +

[castorcharly]

Tout dépend de la façon dont vous avez déclarer le curseur.

Si vous avez juste fait :

Code :

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

DECLARE CurEre CURSOR FOR SELECT....

Vous avez créé un curseur dynamique, scrollable, misajourable....
/...
.../
Cependant il est idiot d'utiliser un curseur qui ne peut pas être optimisé et dans votre cas génère des verrous mortels, au lieu d'une requête qui elle peut l'être.

A +

Bonjour Sql Pro,

Code :

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

1
2
DECLARE CurEre CURSOR FOR SELECT....

Oui, c'est bien ce que j'ai fait, donc là il y a verrou si je comprends bien.

Pour le fait que mon code est idiot, c'est fort possible, je n'ai pas votre expérience dans ce langage.
Mais effectivement, je mets à jour la table que je parcours, ainsi que d'autres.
Il y a peut être une meilleure méthode, c'est pour ça que je viens ici.

Le Transacsql est un langage complexe et les documentations touffues.
Il est difficile et long de l'apprendre et d'en connaître toutes les subtilités.

J'ai fait l'acquisition de livre sur le sujet et j'y ai appris pas mal de chose, mais je suis loin d'en appréhender le début du commencement.

[iberserk]

Bonjour,
Postez votre code, nous vous montrerons comme éviter les curseurs...