Code review comment for lp:~james-w/launchpad/archive-job-db
Revision history for this message
| Stuart Bishop (stub) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
73416ba
(Get the code!)
Generally fine.
patch-2207-64-0.sql
I think the job foreign key constraint should be ON DELETE CASCADE to make job garbage collection a little easier:
job integer NOT NULL CONSTRAINT archivejob__job__fk REFERENCES job ON DELETE CASCADE
(the archive foriegn key constraint should not be).