PO import queue hits a problem with product and distroseries count

Bug #423150 reported by Данило Шеган
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Jeroen T. Vermeulen

Bug Description

We are using shortlist with no parameters to fetch all product and distro series in translation import queue.

This fails with:

 /srv/launchpad.net/production/launchpad-rev-8334/lib/lp/translations/model/translationimportqueue.py:1060: UserWarning: shortlist() should not be used here. It's meant to listify sequences with no more than 15 items. There were 17 items.
  distinct=True))

We have a few products with more than 15 series, and some are close. We should obviously fix this to not be broken in this way.

I will be investigating some more and seeing if this is a critical problem or not.

Related branches

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

It's not the number of productseries that's breaking this shortlist, but the number of distinct products that have Approved entries in the queue.

What I don't get is why this isn't breaking the UI as well: see TranslationImportTargetVocabularyFactory. That uses getRequestTargets too, and as far as I can see, in ways that should return even more products.

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

Ah, it's not actually breaking; it's logging warnings. Though that may still affect things somehow.

Changed in rosetta:
status: Triaged → In Progress
Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

Fixed in devel 9352

Changed in rosetta:
assignee: nobody → Jeroen T. Vermeulen (jtv)
status: In Progress → Fix Committed
Changed in rosetta:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.