bugmessage.index is not populated

Bug #704446 reported by Robert Collins
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Critical
Robert Collins

Bug Description

BugMessage.index exists in the schema to let BugTask:+index be more efficient, but its not populated.

To populate it we need a multistep process:
 step 1: a garbo hourly job that does a small amount of conversions. It needs to do at least 175 per garbo invocation if we want a 6 month completion date. I think that most bugs are small enough we could even set the minimum to 350 : the auto loop tuner will do more if more is acceptable given db load and time taken.
  --- Done

 step 2: start setting BugMessage.index when adding BugMessages - this will require calculating and setting the BugMessage.index for all other messages on the bug at the same time for consistency.
  --- Done

 step 3: when fully populated, make the bugmessage.index column NON-NULL, remove the garbo job to set bugmessage.index and remove the calculation of bugmessage.index for existing messages when adding a new message.
  --- Done

 step 4: We can fix bug 607935

Related branches

Changed in launchpad:
assignee: nobody → Robert Collins (lifeless)
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :
Changed in launchpad:
milestone: none → 11.02
tags: added: qa-needstesting
Changed in launchpad:
status: Triaged → Fix Committed
Revision history for this message
Robert Collins (lifeless) wrote :

step 1 runs fine on qastaging (after we sorted a bunch of qastaging yaks out. Thanks spm!). Its doing > 175 - a lot more. So we'll be able to move forward soon.

tags: added: qa-ok
removed: qa-needstesting
Revision history for this message
Robert Collins (lifeless) wrote :

On staging one run of garbo hourly populates:

launchpad_qastaging=> select count( bugmessage.id ) from bugmessage where index is not null limit 50;
 count
-------
 68532
(1 row)

William Grant (wgrant)
Changed in launchpad:
status: Fix Committed → Fix Released
Changed in launchpad:
status: Fix Released → Triaged
milestone: 11.02 → none
description: updated
Revision history for this message
Robert Collins (lifeless) wrote :

We're 90% through the indexing process; I'm going to start prepping the next phase now.

Revision history for this message
Robert Collins (lifeless) wrote :

200K left to index.

Revision history for this message
Robert Collins (lifeless) wrote :

Completely indexed.

Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

Fixed in stable r12365 (http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/12365) by a commit, but not testable.

Changed in launchpad:
milestone: none → 11.03
tags: added: qa-untestable
removed: qa-ok
Changed in launchpad:
status: Triaged → Fix Committed
tags: added: qa-ok
removed: qa-untestable
description: updated
description: updated
Changed in launchpad:
status: Fix Committed → In Progress
Revision history for this message
Robert Collins (lifeless) wrote :

bug 607935 can now be worked on: the live code will always set bugmessage.index and reindexes if needed before doing anything else. There are nounindexed bugmessages.

tags: removed: qa-ok
Revision history for this message
William Grant (wgrant) wrote :

"always" except for cocoplum and germanium, who add bugmessages but are outside nodowntime.

Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 704446] Re: bugmessage.index is not populated

On Tue, Feb 15, 2011 at 11:09 AM, William Grant <email address hidden> wrote:
> "always" except for cocoplum and germanium, who add bugmessages but are
> outside nodowntime.

oh grrrrrrrrr

Matt, can I have a downtime window for these two services please, just
a simple deploy of the new code which will keep this column intact.

Revision history for this message
Launchpad QA Bot (lpqabot) wrote :

Fixed in stable r12393 (http://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable/revision/12393) by a commit, but not testable.

tags: added: qa-untestable
Changed in launchpad:
status: In Progress → Fix Committed
William Grant (wgrant)
Changed in launchpad:
status: Fix Committed → Fix Released
description: updated
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.