POFile:+translate Timeout error trying to access the translation interface
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | Launchpad itself |
High
|
Данило Шеган | ||
Bug Description
Hi to all,
I am writing to report a problem we encountered on the project that we are administering: logicaldoc.
There are approximately 2 (two) days that are no longer able to run the translations interface on logicaldocTrunk.
I am attaching a screenshot of the problem detected.
In practice, we are no longer able to access the maintenance interface of the single translation of branch Trunk.
To reproduce the problem simply try to access the page for translate german language of logicaldoc (Trunk)
http://
On the other branches, and packages the translation system works.
Note: on Saturday and Sunday we uploaded some single translation files in .po format (on Trunk)
Could you kindly check this ?
Thanks in advance for your effort :-)
OOPS-1752H707 OOPS-1755ED1033
Related branches
- Abel Deuring (community): Approve (code) on 2010-10-26
-
Diff: 335 lines (+91/-20)11 files modifiedlib/lp/translations/doc/potmsgset.txt (+17/-1)
lib/lp/translations/model/potmsgset.py (+4/-17)
lib/lp/translations/stories/standalone/xx-pofile-translate-alternative-language.txt (+8/-0)
lib/lp/translations/stories/standalone/xx-pofile-translate-dismiss-suggestions.txt (+10/-0)
lib/lp/translations/stories/standalone/xx-pofile-translate-legal-warning.txt (+8/-0)
lib/lp/translations/stories/standalone/xx-pofile-translate.txt (+8/-2)
lib/lp/translations/stories/standalone/xx-translationmessage-translate.txt (+10/-0)
lib/lp/translations/stories/translationgroups/60-translation-suggestions.txt (+5/-0)
lib/lp/translations/tests/test_potmsgset.py (+7/-0)
lib/lp/translations/tests/test_suggestions.py (+6/-0)
lib/lp/translations/tests/test_translatablemessage.py (+8/-0)
- Brad Crittenden (community): Approve (code) on 2010-10-27
-
Diff: 24 lines (+3/-4)1 file modifiedlib/lp/translations/model/potmsgset.py (+3/-4)
- Edwin Grubbs (community): Approve (code) on 2010-10-27
-
Diff: 674 lines (+102/-182)10 files modifiedlib/lp/translations/browser/pofile.py (+3/-22)
lib/lp/translations/browser/tests/translationmessage-views.txt (+27/-15)
lib/lp/translations/browser/translationmessage.py (+17/-44)
lib/lp/translations/doc/canonical_url_examples.txt (+4/-4)
lib/lp/translations/doc/potmsgset.txt (+7/-29)
lib/lp/translations/doc/translationmessage.txt (+13/-12)
lib/lp/translations/interfaces/potmsgset.py (+8/-8)
lib/lp/translations/model/potmsgset.py (+7/-11)
lib/lp/translations/model/translationmessage.py (+0/-7)
lib/lp/translations/tests/test_potmsgset.py (+16/-30)
- Tim Penhey (community): Approve (code) on 2010-10-28
- Launchpad code reviewers: Pending (code) requested 2010-10-27
-
Diff: 112 lines (+19/-19)3 files modifiedlib/lp/translations/browser/pofile.py (+4/-9)
lib/lp/translations/browser/tests/test_translationmessage_view.py (+1/-2)
lib/lp/translations/browser/translationmessage.py (+14/-8)
- Gavin Panella (community): Approve on 2010-11-11
-
Diff: 88 lines (+47/-13)2 files modifiedlib/lp/translations/model/translationmessage.py (+17/-13)
lib/lp/translations/tests/test_translationmessage.py (+30/-0)
| Alessandro Gasparini (blucecio74) wrote : | #1 |
| summary: |
- Timemout error trying to access the translation interface + Timeout error trying to access the translation interface |
| affects: | launchpad → rosetta |
| description: | updated |
| Changed in rosetta: | |
| status: | New → Triaged |
| importance: | Undecided → High |
| description: | updated |
| Данило Шеган (danilo) wrote : | #3 |
Also, note that you can manually reduce the batch size for now to work-around the problem. I.e. append "?batch=3" to the URL to see only 3 messages for translation per page.
Fixed in stable r11800 <http://
| Changed in rosetta: | |
| assignee: | nobody → Jeroen T. Vermeulen (jtv) |
| milestone: | none → 10.11 |
| tags: | added: qa-needstesting |
| Changed in rosetta: | |
| status: | Triaged → Fix Committed |
stable r11800 is in db-stable r9929.
Fixed in stable r11807 <http://
Successfully Q/A'ed Danilo's branch as well as the first two of mine:
lp:~danilo/lp-production-configs/disable-global-suggestions (lp-production-
lp:~jtv/launchpad/bug-662552-suggestive-potemplates (devel 11800)
lp:~jtv/launchpad/bug-662552-defer-potmsgset-filter (devel 11807)
Fixed in stable r11811 <http://
Marked as qa-ok so as not to block deployments.
| tags: |
added: qa-ok removed: qa-needstesting |
| Jeroen T. Vermeulen (jtv) wrote : | #11 |
This branch was also good in Q/A:
lp:~jtv/launchpad/bug-662552-get-tm-or-dummy (devel 11811)
Fixed in stable r11818 <http://
| tags: |
added: qa-needstesting removed: qa-ok |
Having trouble, probably with this branch:
lp:~jtv/launchpad/bug-662552-view-permissions (devel 11818)
Logged in as an unprivileged user, I can't enter suggestions where I should be able to.
| Jeroen T. Vermeulen (jtv) wrote : | #14 |
Phew. It's okay. I was logged in as an unprivileged user with the licensing agreement declined. The branch is OK.
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in rosetta: | |
| status: | Fix Committed → Fix Released |
Fixed in stable r11818 <http://
| tags: |
added: qa-needstesting removed: qa-ok |
| Changed in rosetta: | |
| status: | Fix Released → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
Note that these fixes are all incremental, and don't really address the problem. Problem is only worked around for now by disabling global suggestions.
The problem seems to come from the pofile derivation code which gets executed too many times (TranslationMes
| Данило Шеган (danilo) wrote : | #17 |
Ok, I think I figured where the problem is: we call ensureBrowserPOFile for every plural form for each of imported and shared translations. For those messages, we don't even have to figure it out, because we can consider them "local" to this PO file, so just using self.pofile in the TM view is fine. I'll be testing the hypotheses on staging first.
| Данило Шеган (danilo) wrote : | #18 |
This's apparently not enough. More investigation needed.
| Changed in rosetta: | |
| assignee: | Jeroen T. Vermeulen (jtv) → Данило Шеган (danilo) |
| status: | Fix Committed → In Progress |
| Данило Шеган (danilo) wrote : | #19 |
Ok, I've managed to speed up the relevant query roughly a 100x by knowing more about the data model than postgres can :) I'll just land a simplistic fix which doesn't reduce the query count.
Fixed in stable r11911 <http://
| tags: |
added: qa-needstesting removed: qa-ok |
| Changed in rosetta: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| summary: |
- Timeout error trying to access the translation interface + POFile:+translate Timeout error trying to access the translation + interface |
| Changed in rosetta: | |
| status: | Fix Committed → Fix Released |
| Данило Шеган (danilo) wrote : | #21 |
We need to re-enable global suggestions before we can be certain this works. Here's the MP for that: https:/
It seems fetching global suggestions is taking even longer than usual (probably related to PG 8.4 upgrade). Though, fetching POFiles takes a long time as well. As a stop-gap measure, we are disabling global suggestions so we've got more room for investigation (and so we don't block users while we do the investigation).