Merge lp:~danilo/launchpad/bug-814580-cold-db into lp:launchpad
Proposed by
Данило Шеган
Status: | Rejected |
---|---|
Rejected by: | Данило Шеган |
Proposed branch: | lp:~danilo/launchpad/bug-814580-cold-db |
Merge into: | lp:launchpad |
Prerequisite: | lp:~danilo/launchpad/bug-814580 |
Diff against target: |
54 lines (+20/-9) 3 files modified
database/schema/comments.sql (+8/-8) database/schema/patch-2208-99-0.sql (+11/-0) lib/lp/translations/model/translationsharingjob.py (+1/-1) |
To merge this branch: | bzr merge lp:~danilo/launchpad/bug-814580-cold-db |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Robert Collins (community) | Needs Resubmitting | ||
Stuart Bishop | Pending | ||
Review via email: mp+69981@code.launchpad.net |
Description of the change
= Bug 814580: post-cleanup =
This renames the table PackagingJob to TranslationShar
This requires a downtime deployment since all appservers have to be restarted after this change is deployed.
== Proposed fix ==
To post a comment you must log in.
Unmerged revisions
- 13583. By Данило Шеган
-
Merged bug-814580 into bug-814580-cold-db.
- 13582. By Данило Шеган
-
Add a DB patch to rename PackagingJob table to TranslationShar
ingJob.
This will need approval from me or francis as we no long have routine coordinated downtime + schema + new appserver versions.
Whats the reason for needing a deploy-to-both ?
I'm guessing its the table rename? If so, I suggest you create a view that just presents the table, so that the same appserver code will work. Then once the appservers are upgraded, we can drop the view.