Ticket #104 (new defect)

Opened 1 year ago

Last modified 1 year ago

adding branch= to the waterfall is expensive

Reported by: warner Assigned to: warner
Priority: critical Milestone: 0.8.0
Component: statusplugins-web Version: 0.7.5
Keywords: waterfall, branches Cc:

Description

I added a max_search= to the IBuilderStatus.generateFinishedBuilds method, to limit how many builds are searched when constructing the one_line_per_build and one_box_per_builder pages, but the eventGenerator() method used by the waterfall has no such limit. As a result, actually using the new branch= argument on the waterfall page is very expensive, as any Builder which has never built that branch will cause eventGenerator() to run through its entire history, pulling each build off disk to check the branch name.

The long-term fix for this is to use a database for status information which can get us this information faster. The short-term fix (after 0.7.6, unfortunately) is to add a similar max_seach= argument to eventGenerator. It's too complex to add today, though.

Change History

12/22/07 00:33:34 changed by warner

  • milestone changed from 0.7.7 to 0.7.8.

No progress made on this yet, bumping to 0.7.8