Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(Orchestrator): now aborts outdated tasks #1036

Merged
merged 1 commit into from
Aug 26, 2024
Merged

Conversation

nicoprow
Copy link
Contributor

@nicoprow nicoprow commented Aug 19, 2024

Description

In this pull request the Orchestrator now aborts tasks with outdated business partner data information from the sharing members. On creating new tasks the Gate will check for currently pending tasks for the same record and abort them.

An aborted task counts as a finished tasks and will be removed by the retention timeout check.

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

@nicoprow nicoprow force-pushed the feat/orchestrator-abort branch 2 times, most recently from 3ca6d81 to 130a9cd Compare August 19, 2024 10:20
@nicoprow nicoprow marked this pull request as ready for review August 19, 2024 10:49
Copy link
Contributor

@SujitMBRDI SujitMBRDI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, need to resolve conflicts.

- on creating new tasks the Gate will check for currently pending tasks for the same record and abort them
@nicoprow nicoprow force-pushed the feat/orchestrator-abort branch from 130a9cd to 4b36fd3 Compare August 23, 2024 08:23
@nicoprow nicoprow merged commit ecd3108 into main Aug 26, 2024
17 checks passed
@nicoprow nicoprow deleted the feat/orchestrator-abort branch August 26, 2024 04:47
@nicoprow nicoprow added this to the BPDM v6.2.0 / R24.12 milestone Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants