Closed shivamkm07 closed 9 months ago
The PR validation is failing probably because the respective PR in durabletask-protobuf https://github.com/microsoft/durabletask-protobuf/pull/20 is not merged yet. Should pass after that
@shivamkm07 the https://github.com/microsoft/durabletask-protobuf/pull/20 PR is merged. Please update the submodule reference in this PR so we can get the CI passing. I'll then proceed to sign off and merge this PR.
@shivamkm07 the microsoft/durabletask-protobuf#20 PR is merged. Please update the submodule reference in this PR so we can get the CI passing. I'll then proceed to sign off and merge this PR.
Done
This PR fixes issues in current implementation of Cascade Terminate and adds support for cascade Purge. It is required to support https://github.com/dapr/dapr/issues/6393. As per current implementation,
Each language SDK must implement the cascade terminate feature
A failed sub-orchestration in the chain might block termination of its own downstream sub-orchestrations
This PR moves the recursive termination logic to backend side and so no SDK would need to have any implementation. Also it fixes the issue of failed orchestration blocking termination of its downstream sub-orchestrations.It also adds a new test to validate the both scenarios. (The second test added failed for the original implementation.)
This PR also implements Cascade Purge, similar to Cascade terminate and adds tests for the same. The Proto updates are corresponding to the protobuf updates: https://github.com/microsoft/durabletask-protobuf/pull/20
Fixes: #25