Still some timeouts on DistroSeriesLanguage

Bug #400494 reported by Jeroen T. Vermeulen on 2009-07-17
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Medium
Unassigned

Bug Description

After the fix for bug 327100, we're still seeing some DistroSeriesLanguage:+index timeouts on edge.

For example, oops 1293EB63 seems to show us being bogged down selecting 300 SourcePackageNames. A strange thing there is that query #25 seems to take over 3 seconds according to the log, which should make it top the long-queries list but doesn't. Client-side latency is also considerable (almost 10 seconds!) but the major chunk is still SQL time.

Oops 1293EB35 and oops 1293EA307 on the other hand seem to spend all their time client-side. Again, batch size is 300 and non-SQL time is just under 10 seconds.

It'd be nice if we could prejoin or batch-fetch those SourcePackageNames, but the main thing seems to be getting that non-SQL time down.

If the bulk of it is spent marshalling objects, we'd have to stop using POTemplate.name on that page. Instead we'd have to query the template name directly. Composing links to POFiles and such may become ugly: presumably the POTemplate and SourcePackageName are implicitly queried every time we try to link to a POFile.

tags: added: timeout

Looking at OOPS-1293EA307, it seems a simple fix in the style of our performance weeks would improve the situation in a big way:

  https://pastebin.canonical.com/19940/

I'll try cowboying this later and checking if it helps.

Changed in rosetta:
status: New → Triaged
importance: Undecided → Medium
Jeroen T. Vermeulen (jtv) wrote :

The timeouts we were still seeing on edge may have had something to do with the then-ongoing Storm upgrade. Now, on production with the 2.2.7 changes, they occur so rarely that they don't show up in any of the oops reports' top-10 lists.

Jeroen T. Vermeulen (jtv) wrote :

Looks like the fix for 515702 took care of the last timeouts here.

Changed in rosetta:
milestone: none → 10.02
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers