Discussion :

[nessaure]

Bonjour à tous,

je me permets de vous contacter car j'ai un problème de performance avec l'exécution d'une procédure stockée.

Actuellement,j'ai une procédure qui prend en entrée plusieurs paramètres:
- @Id INT,
- @MD MD READONLY,
- @Ids NVARCHAR(MAX) OUTPUT

Cette procédure a pour but de traiter les données de la table valued @MD.
Pour chaque élément de cette table valued, je fais une boucle while dans laquelle je fais plusieurs traitement:
- Un ensemble de select sur ma base de données. Suivant les résultats j'affecte à des variables des valeurs différentes
- Une insertion dans une table de ma base avec les valeurs récupérées juste avant.
- Je récupère la clé primaire à l'aide de SET @Tid = (SELECT @@IDENTITY)
- Dans ma variable @Ids de sortie je concatène les clés primaires séparées par une virgule
SET @Ids = (SELECT CONCAT(@Ids, @Tid, ','))

En résumé pour chaque ligne de ma table valued je fais un certain nombre de traitements puis je fais une insertion en base.

Je pense que les problèmes de performances peuvent venir du fait que j'insère ligne par ligne au lieu de faire un seul insert.
Pensez vous que cela vient de ça?

De plus, si ça peut venir de là, je ne vois pas comment faire une seule insertion, étant donné que je dois pourvoir retourner l'ensemble des clés primaires créées.
Avez vous une idée?

Je vous remercie d'avance.

[Invité]

Montre nous directement le code, parce qu'optimiser de l'abstraction, ça risque d'être sexy !
Est-ce qu'il y a des curseurs dedans ?

[aieeeuuuuu]

Bonjour

De plus, si ça peut venir de là, je ne vois pas comment faire une seule insertion, étant donné que je dois pourvoir retourner l'ensemble des clés primaires créées.

Pour ça, vous pouvez utiliser la clause OUTPUT.

Pour l'optimisation, comme déjà dit, difficile d'optimiser du code... sans avoir le code !

[SQLpro]

Je pense que les problèmes de performances peuvent venir du fait que j'insère ligne par ligne au lieu de faire un seul insert.
Pensez vous que cela vient de ça?

Dans tous les cas, OUI !

De plus, si ça peut venir de là, je ne vois pas comment faire une seule insertion, étant donné que je dois pourvoir retourner l'ensemble des clés primaires créées.

La solution : formation + imagination !!!

Avez vous une idée?

OUI !

A +

[julien94320]

Bonjour,

Avec le code c'est mieux ...

Quelques pistes :

Etant donné que vous utilisez beaucoup de paramètres j'essaierais d'ajouter un Hint : option (recompile) après chaque select afin que l'optimiseur cherche à effectuer les requetes avec un plan correcte et non pas imaginé avec des valeurs fausses en se basant sur la dernière execution qui s'est dérouler avec d'autres parametre et dont les critères de recherches sont completements différents ...

Essayez de mettre a jour vos statistiques aussi pour voir ...

[nessaure]

Bonjour à tous,

merci pour vos réponses. Je n'utilise pas de curseurs, et je vais essayer de regarder vos pistes.
Si je n'y arrive pas, j'essayerai de mettre un copié/collé du code, mais la procédure est très longue (environ 500 lignes ...).

Dans tous les cas, OUI !



La solution : formation + imagination !!!



OUI !

A +

Super ! Merci pour cette construction, j'espère que ça ne t'a pas pris trop de temps, je m'en voudrais.
Au passage formation + imagination = carnage très souvent :-)

[Invité]

Si je n'y arrive pas, j'essayerai de mettre un copié/collé du code, mais la procédure est très longue (environ 500 lignes ...).

Euh, relativisons quand même, 500 lignes de codes commentés, c'est juste une procédure un peu longue mais pas très longue.

[nessaure]

Euh, relativisons quand même, 500 lignes de codes commentés, c'est juste une procédure un peu longue mais pas très longue.

Ah désolé je ne connais pas bien ; pour moi ça me semble très long :-)