Ticket #1813 (closed enhancement: fixed)

Opened 2 years ago

Last modified 2 years ago

Grid and T-Grid performance

Reported by: aivarsk Owned by: ayust
Priority: major Milestone: undecided
Version: 0.8.3p1 Keywords:
Cc:

Description

Instead of collecting all unique SourceStamps? from all builders and then taking last numBuilds sourcestamps, the code now takes only last numBuilds Builds/SourceStamps? from each builder.

This improves performance from 15 seconds with default caching options (18 builders, 1100 builds) to under a second.

Attachments

buildbot-grid.diff Download (3.5 KB) - added by aivarsk 2 years ago.

Change History

Changed 2 years ago by aivarsk

comment:1 Changed 2 years ago by ayust

  • Owner set to ayust
  • Status changed from new to assigned

Note to self, integrate this on Tuesday.

comment:2 Changed 2 years ago by Amber Yust

  • Status changed from assigned to closed
  • Resolution set to fixed

Improve grid/tgrid performance.

Patch diff submitted by alvarsk (Closes #1813):

Instead of collecting all unique SourceStamps? from all builders and then taking last numBuilds sourcestamps, the code now takes only last numBuilds Builds/SourceStamps? from each builder.

This improves performance from 15 seconds with default caching options (18 builders, 1100 builds) to under a second.

Changeset: 074428d5a2dd395293e41d82739bca6a5bb08aff

Note: See TracTickets for help on using tickets.