The prior implementation seemed to be a bit unintuitive with how cancellation was implemented and used by the voter job. This PR is attempting to make the behavior as explicit as possible.
How this works
Rather than having Execute and Cancel function that are called, there is a single Execute which takes in the results of resolving the dependencies.
Why this should be merged
The prior implementation seemed to be a bit unintuitive with how cancellation was implemented and used by the
voterjob. This PR is attempting to make the behavior as explicit as possible.How this works
Rather than having
ExecuteandCancelfunction that are called, there is a singleExecutewhich takes in the results of resolving the dependencies.How this was tested