Discussion :

[Jinroh77]

Bonjour,
Je ne sais pas si mon titre est très clair amis je vais m'expliquer au mieux.
pour une PS, j'ai un paramètre @CodeAction (correspondant à une valeur d'un champs de la table Action) qui peut arriver NULL.
Je souhaite :
- si le paramètre est NULL, ne pas le prendre en compte
- sinon faire le croisement.

L'idée est donc d'utiliser

Code :

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

WHERE Action.CodeAction = ISNULL(@CodeAction, Action.CodeAction)

Dans ce cas il fait ce que je demande.
Le problème est que cette table Action est très lourde et le temps de traitement en souffre.

la solution rustine : décomposer le select, mais ce n'est vraiemt pas bon à maintenir..

Code :

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

1
2
3
4
5
6
7
8
9
IF @CodeAction IS NOT NULL
SELECT ...
FROM ...
WHERE act.CodeAction = @CodeAction
 
ELSE
SELECT ...
FROM ...
WHERE <rien>

Niveau maintenance ça fait vraiment dégueulasse mais au moins c'est beaucoup plus rapide.

Merci pour vos conseils.

[Jinroh77]

Après quelques essais et recherches supplémentaires,
j'ai remplacé le ISNULL par

Code :

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

act.CodeAction = COALESCE(@CodeAction, @CodeAction, act.CodeAction)

Cela semble s'executer beaucoup plus vite que le ISNULL (qui commencait à prendre plus de 15min.... là je suis plus de l'ordre des 3s.

Est-ce normal un tel gain de temps ou est ce qu'il y a quelque chose de mauvaise qui se ballade sur cette base ?

Merci

[SQLpro]

Code :

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

act.CodeAction = COALESCE(@CodeAction, act.CodeAction)

suffira !

Evicemment que cela va plus vite :
Avec un OR aucun index ne peut être activé.
Avec cette condition votre prédicat devient "sargable" et donc le moteur peut utiliser un index.

A +

[Jinroh77]

EUh... je n'ai aps tout compris à l'explication.

Ok pour le OR, je ne le savais pas mais c'est bon à savoir : utiliser le moins possible.
Par contre je n'ai pas compris comment le prédicat devient sargable avec le COALESCE et pas avec le ISNULL ?

Merci

[escartefigue]

Pour qu'un prédicat soit Sargable (Search Argument Able) donc indexable, il faut que les opérandes soient de même format
COALESCE le permet puisque cette fonction prend la première valeur non nulle de la liste

[mikedavem]

Quel sont les types exactes de ta variable @CodeAction et colonne act.CodeAction ?

Il y a en effet une différence entre ISNULL et COALESCE sur la manière de traiter les types de données . COALESCE prendra le type avec la plus haute précédence alors que ISNULL prendra le premier type venu.

Tu peux fournir les 2 plans d'exécutions dans ton cas ?

++