1017117 - Add cron job for prune the last visited table

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Infrastructure, defect)

Description

[Dylan Hardison [:dylan] (he/him)]

Bug 1000917 adds a database table which needs to be pruned nightly.

Can you please add a cron job to run bugzilla/clean-bug-user-last-visit.pl each night. This job needs full database access it probably makes sense to run it on the same system that currently runs whine.pl, etc.

this file doesn't currently exist on production, however it will be deployed in during the next push.

Comment 1

[Philippe M. Chiasson (:gozer)]

Looking at the script, it runs only one query:

DELETE FROM
    bug_user_last_visit 
WHERE 
    last_visit_ts < NOW() - Bugzilla->params->{last_visit_keep_days} DAY

Can I correctly assume there is an index on the last_visit_ts column?

Comment 2

[Dylan Hardison [:dylan] (he/him)]

There is not an index on that column currently, am I to understand that would be preferable before it is deployed? If so I can do that.

Comment 3

[:glob ✱]

(In reply to Dylan William Hardison [:dylan] from comment #2)
> There is not an index on that column currently, am I to understand that
> would be preferable before it is deployed? If so I can do that.

file an upstream bugzilla bug that modifies Bugzilla/Install/DB.pm.

Comment 4

[Dylan Hardison [:dylan] (he/him)]

Filed as bug 1022923

Comment 5

[:glob ✱]

ping?

Comment 6

[Dylan Hardison [:dylan] (he/him)]

This is ready to be switched on in production, right?

Comment 7

[:glob ✱]

(In reply to Dylan William Hardison [:dylan] from comment #6)
> This is ready to be switched on in production, right?

yes.  it was ready at the time the bug was filed.
bug 1022923 was an optimisation, not a blocker.

Comment 8

[Kendall Libby [:fubar] (he/him)]

Added in r90256. Will run daily at midnight.