Make export script use slave store
Bug #286237 reported by
Jeroen T. Vermeulen
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
Jeroen T. Vermeulen | ||
Bug Description
The export script barely does any writing to the database (AFAIR it only removes the requests it's completed from the queue). Thus it could use the slave store and benefit from replication.
Not to be confused with bug 286236, which is about the language pack exporter.
Related branches
lp:~jtv/launchpad/bug-286237
- Guilherme Salgado (community): Approve
- Canonical Launchpad Engineering: Pending (code) requested
-
Diff: 754 lines (+279/-94)14 files modifiedlib/lp/translations/browser/tests/test_baseexportview.py (+3/-2)
lib/lp/translations/doc/poexport-queue.txt (+6/-5)
lib/lp/translations/doc/poexport-request-productseries.txt (+3/-2)
lib/lp/translations/doc/poexport-request.txt (+8/-3)
lib/lp/translations/doc/poexportqueue-replication-lag.txt (+89/-0)
lib/lp/translations/doc/potmsgset.txt (+12/-8)
lib/lp/translations/interfaces/poexportrequest.py (+23/-7)
lib/lp/translations/interfaces/potmsgset.py (+14/-8)
lib/lp/translations/model/poexportrequest.py (+68/-31)
lib/lp/translations/model/potmsgset.py (+8/-1)
lib/lp/translations/scripts/po_export_queue.py (+10/-24)
lib/lp/translations/tests/test_potmsgset.py (+23/-1)
lib/lp/translations/tests/test_suggestions.py (+8/-1)
lib/lp/translations/tests/test_translatablemessage.py (+4/-1)
| Changed in rosetta: | |
| importance: | Undecided → Low |
| status: | New → Triaged |
| Changed in rosetta: | |
| milestone: | none → 10.03 |
| assignee: | nobody → Jeroen T. Vermeulen (jtv) |
| status: | Triaged → In Progress |
| Changed in rosetta: | |
| status: | In Progress → Fix Committed |
| tags: | added: qa-ok |
Revision history for this message
| Ursula Junque (ursinha) wrote Bug fixed by a commit | #1 |
| tags: |
added: qa-needstesting removed: qa-ok |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in rosetta: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r10435 <http://