Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#2645 closed task (fixed)

[nine] Implement Request Collapsing

Reported by: dustin Owned by: delanne
Priority: major Milestone: 0.9.0
Version: Keywords:
Cc:

Description

*Currently Broken*: Builds for merged requests currently only refer to one of the merged requests.

  • Before a build request is added to a non-empty queue, examine each unclaimed build request in that queue. If it is compatible with the new request (defined as having matching sourcestamps, except for revision), then claim the old request and immediately complete it with result SKIPPED.
  • Allow the same kind of configuration as for merging: global and per-builder, with options True (merge), False (never), and callable (called with two build request dictionaries, and expected to get the rest from the data API).
  • Call it "queue collapsing" rather than merging (hat-tip to Mozilla release engineering for the term)

Note that the first bullet here assumes that a more recently-submitted request should be preferred over an older request.

See https://plus.google.com/105883044168332773236/posts/TG8DHus4L4D

Change History (4)

comment:1 Changed 5 years ago by delanne

  • Owner set to delanne
  • Status changed from new to accepted

comment:3 Changed 5 years ago by delanne

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

PR1261 was merged. Close this ticket.

comment:4 Changed 5 years ago by Mikhail Sobolev <mss@…>

In 8f7ec588b179daecbbb9021f876c5154ce456ccd:

remove a note about non-working request collapsing

See ticket:2645

Note: See TracTickets for help on using tickets.