Discussion :

[BLJ.CHAUVIN]

Bonjour à tous,

J'ai un problème de durée d'exécution trop longue avec un trigger before delete.
"Fatal error: Maximum execution time of 300 seconds exceeded"
Grosso modo j'ai une table 'news', une table 'usager' et une table 'historique', quand j'efface une news je souhaite que mon trigger efface automatiquement les lignes correspondant à cette news dans ma table 'historique'.



J'ai essayé de modifier mon trigger, avec ou sans jointure, mais le résultat est toujours le même.
Voici les contraintes de mes tables:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
-- Contrainte pour la table 'news'
ALTER TABLE `news`
  ADD CONSTRAINT `FK_news_usager` FOREIGN KEY (`idusager`) REFERENCES `usager` (`idusager`);
 
-- Contraintes pour la table `historique`
ALTER TABLE `historique`
  ADD CONSTRAINT `FK_histo_adresse` FOREIGN KEY (`idadresse`) REFERENCES `adresse` (`idadresse`),
  ADD CONSTRAINT `FK_histo_annuaire` FOREIGN KEY (`idannuaire`) REFERENCES `annuaire` (`idannuaire`),
  ADD CONSTRAINT `FK_histo_catref` FOREIGN KEY (`idcatref`) REFERENCES `catref` (`idcatref`),
  ADD CONSTRAINT `FK_histo_contact` FOREIGN KEY (`idcontact`) REFERENCES `contact` (`idcontact`),
  ADD CONSTRAINT `FK_histo_ficheref` FOREIGN KEY (`idficheref`) REFERENCES `ficheref` (`idficheref`),
  ADD CONSTRAINT `FK_histo_missionref` FOREIGN KEY (`idmissionref`) REFERENCES `missionref` (`idmissionref`),
  ADD CONSTRAINT `FK_histo_news` FOREIGN KEY (`idnews`) REFERENCES `news` (`idnews`),
  ADD CONSTRAINT `FK_histo_usager` FOREIGN KEY (`idmodificateur`) REFERENCES `usager` (`idusager`);

La dernière version de mon Trigger est celle-ci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
DROP TRIGGER IF EXISTS deleteNewsHisto;
DELIMITER //
CREATE TRIGGER deleteNewsHisto 
BEFORE DELETE ON news
FOR EACH ROW
BEGIN
DELETE FROM historique 
INNER JOIN news 
ON news.idnews = historique.idnews
WHERE historique.idnews = news.OLD.idnews;
END
//
DELIMITER ;

Si vous voyez ce qui ne va pas, je vous écoute d'une oreille attentive.

Question supplémentaire: si je veux ajouter un delete à ce trigger pour qu'il efface en plus les occurences liées à ma news dans une autre table, est-ce possible? (je veux dire sans écrire un autre trigger mais en ajoutant seulement cette instruction à mon trigger existant).

[Yanika_bzh]

Petite question, a quoi sert votre jointure dans le delete du trigger ?

[BLJ.CHAUVIN]

Comme je le disais : "j'ai testé pas mal de modification sur mon trigger, du plus logique au plus farfelu... en désespoir de cause."

Donc la réponse est: surement à rien mon capitaine! Mais sans cette jointure il ne fonctionne malheureusement pas mieux.

[Yanika_bzh]

idnews dans votre table historique est bien une clé primaire ?

[BLJ.CHAUVIN]

Non dans la table 'historique' le champ 'idnews' est en clé étrangère pas primaire. 'idnews' est la clé primaire de la table 'news'.

[Yanika_bzh]

mettez urgemment un index sur cette colonne alors

[BLJ.CHAUVIN]

Vous me suggérez de mettre un index sur chaque clé étrangère de la table 'historique' ?

Ceci devrait en effet diminuer le temps de recherche lors de mes requêtes (et je vous remercie pour ce conseil) mais cela ne devrait pas avoir d'incidence sur le "non-fonctionnement" de mon trigger, n'est-il pas?

[Yanika_bzh]

Maximum execution time of 300 seconds exceeded

Si vous diminuez de manière significative le traitement de vos requêtes, vous ne devriez pas tomber dans ce timeout.

[BLJ.CHAUVIN]

Si vous diminuez de maniere significative le traitement de vos requetes, vous ne devriez pas tomber dans ce timeout.

Il n'y a (pour l'instant) qu'une dizaine de lignes dans ma table historique, je ne crois donc pas que mon problème vient de là. A mon avis mon trigger est mal écrit, c'est ce qui provoque le timeout. Pour une raison conceptuelle ou syntaxique mon trigger ne peut tout simplement pas être lu.

C'est pour ça que j'ai fourni une image de mon MPD et le bout de code correspondant aux contraintes des tables, quelque chose a du m'échapper.

[Yanika_bzh]

pourquoi mettre delimiter dans le script de votre trigger ?

[BLJ.CHAUVIN]

Aucune idée... dans les exemples que j'ai trouvés il y a avait souvent DELIMITER je l'ai donc intégré à mon trigger.

Je viens de testé sans, changement de message d'erreur

Code :

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

1
2
CREATE OR REPLACE TRIGGER deleteNewsHisto BEFORE
DELETE ON news FOR EACH ROW BEGIN DELETE FROM historique WHERE historique.idnews = news.OLD.idnews;

MySQL a répondu:

#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'TRIGGER deleteNewsHisto BEFORE DELETE ON news FOR EACH ROW BEGIN
DELETE FRO' at line 1

[Yanika_bzh]

Je posais la question, car il n'y a pas d'autres instructions dans votre trigger, alors j'ai pensé que le source du trigger n’était pas complet...

[ced]

Bonjour,

Ce que je ne comprends pas bien, c'est pourquoi faire un trigger qui a l'air de faire ce pour quoi le ON DELETE CASCADE d'une clé étrangère est prévu ?
Pourquoi vouloir réinventer la poudre ?

[GyZmoO]

Bonjour,

Petite précision :

Vous me suggérez de mettre un index sur chaque clé étrangère de la table 'historique' ?

Ceci devrait en effet diminuer le temps de recherche lors de mes requêtes (et je vous remercie pour ce conseil) mais cela ne devrait pas avoir d'incidence sur le "non-fonctionnement" de mon trigger, n'est-il pas?

Si vous utilisez des tables InnoDB, pas besoin de créer un index sur les colonnes représentant des foreign keys, InnoDB le fait automatiquement