How do I delete the rows that are 30+ days old?

When I run a cronjob, I want to delete from ajaxfavourites in the exptime column all rows that are 30+ days old. I tried:

"DELETE FROM ajaxfavourites WHERE exptime < NOW() + INTERVAL 30 DAY"

Code (markup):

For some reason it deletes them like they need to be deleted now. I know I am doing something wrong. I don't know if it makes a difference or not the date format in the expire column is 2015-09-19 21:54:24

 

Well, first of all, your comparison checks to see if they're LESS than now + 30 days. Also, you should probably put now + the interval in a ()

 

LOL. I didn't see it. Thanks bro.

 

Another way of doing it:

DELETE FROM `ajaxfavourites` WHERE exptime  < DATE_SUB(curdate(), INTERVAL 30 DAY);

Code (markup):

 

Some advice, store the expiration in the advert as the ACTUAL expiration time instead of during your delete. Adding new ones is going to be run a lot less frequently (you hope) than checking for deletion, and the more complex delete is going to bog things down longer.

so when you INSERT INTO:

exptime = DATE_ADD(NOW(), INTERVAL 30 DAY)

that way your delete query is just:

DELETE FROM ajaxfavorites WHERE exptime < NOW()

Far simpler, makes the larger query faster. Only problem with this approach is that if you want to change the expiration time you'd have to go through and update them all -- which really shouldn't be that big a deal.

Basically, if you're going to store the expiration time, STORE THE EXPIRATION TIME not the CREATED TIME.