Discussion :

[peyau]

Bonjour,

J'aimerais supprimer des données qui datent d'il y a plus de 14j dans ma base de données.
J'ai essayé quelque chose comme ceci :

Code :

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

1
2
3
4
5
6
CREATE EVENT DeleteEveryTwoWeeks
ON SCHEDULE EVERY 1 DAY
STARTS '2019-12-06 08:00:00'
DO
SET SQL_SAFE_UPDATES = 0;
DELETE FROM maTable WHERE DATEDIFF(CURDATE(), date) > 14;

Mais je vois que la suppression dans ma base de données ne se fait pas.

Le SET SQL_SAFE_UPDATES = 0; est là sinon j'ai une erreur

Error Code: 1175. You are using safe update mode and you tried to update a table without a WHERE that uses a KEY column To disable safe mode, toggle the option in Preferences -> SQL Editor and reconnect. 0.000 sec

Et j'ai l'impression de devoir le faire chaque jour pour pouvoir exécuter la requête de suppression.

Y aurait-il une autre solution?
Que faire pour que l'événement se lance correctement, et est-il possible de vérifier si l'événement s'est exécuté grâce à des logs?

Je vous remercie

[escartefigue]

bonjour.

Vu que "date" est un mot réservé SQL, votre requête devrait être rejetée : il faut encadrer cette colonne (si c'est bien son nom) par des quotes inversées puisqu'a priori votre SGBD est MySQL
Le plus simple est d'éviter d'utiliser des mots réservés comme nom d'objet.
Mais vu le message d'erreur, c'est plus probablement qu'aucune colonne ne s'appelle "date" dans votre table

Par ailleurs, le nom de l'event "DeleteEveryTwoWeeks" est mal choisi : il ne va pas se produire une fois toutes les deux semaines, mais tous les jours (ON SCHEDULE EVERY 1 DAY). Par contre, il va supprimer les éléments datant d'au moins deux semaines. Ce n'est pas la même chose

[peyau]

Bonjour,

Effectivement le nom de l'événement n'avait aucun sens, merci de me l'avoir fait remarquer. Je voulais faire autre chose à la base et j'ai complètement oublié de changer le nom de l'événement...

Par contre, je confirme que j'avais bien une colonne nommée "date".
Dans le doute, je l'ai simplement renommée "ddate" pour effectuer un test, et j'ai toujours le message d'erreur si je ne modifie pas cette valeur SET SQL_SAFE_UPDATES = 0; juste avant (non je n'ai pas oublié de modifier "date" par "ddate" dans la requête SQL, je le précise car je sais que ça aurait pu m'arriver )

[escartefigue]

L'event a été modifié, il faut donc publier le nouveau code et le (nouveau ?) message d'erreur associé à son exécution

[peyau]

Code :

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

1
2
3
4
5
6
CREATE EVENT DeleteRecordsOlderThanTwoWeeks
ON SCHEDULE EVERY 1 DAY
STARTS CURRENT_TIMESTAMP  --- current_timestamp seulement pour tester l'événement tout de suite et ne pas attendre demain ou lundi
DO
SET SQL_SAFE_UPDATES = 0;
DELETE FROM maTable WHERE DATEDIFF(CURDATE(), ddate) > 14;

Le message d'erreur reste exactement le même
Je tiens à préciser que j'ai le message d'erreur lorsque j'exécute cette requête seule

Code :

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

DELETE FROM maTable WHERE DATEDIFF(CURDATE(), ddate) > 14;

Je n'ai pas le message d'erreur au moment où j'exécute mon CREATE EVENT

[escartefigue]

Pouvez-vous communiquer également le script complet du CREATE TABLE.

Et que donne l'ordre SELECT équivalent exécuté sur cette même table ? (remplacez juste DELETE par SELECT * pour tester)

[Michel.Priori]

Bonsoir,

Personnellement je remplacerais :

Code :

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

DELETE FROM maTable WHERE DATEDIFF(CURDATE(), ddate) > 14

par

Code :

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

DELETE FROM maTable WHERE  ddate) < dateadd(CURDATE(), INTERVAL -14 DAY)

histoire de ne pas avoir à recalculer l'écart de date à chaque ligne ...