Skip to content

fs.2015_Oct_1

Before, the execution graph wouldn't check to see if a job had already been canceled before canceling it. This meant that even if that job was a dependee of another canceled or failed job, it would still end up in the queue. The result is that the queue could grow to be very large with deep graphs, containing mostly duplicate cancelations.

This patch changes the cancellation handling so that the status table is checked for each dependee before canceling it. It also marks them as queued so that the status table knows that they are in progress.

Testing Done:
Wrote a regression test, made it pass. CI away on the PR.

Bugs closed: 2290

Reviewed at https://rbcommons.com/s/twitter/r/2910/

closes #2290
closes #2282
Assets 2
Loading