Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Split event system task implementation between start and execute #3187

Merged
merged 2 commits into from
Aug 18, 2022

Conversation

apanicker-nflx
Copy link
Collaborator

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes (Please run ./gradlew generateLock saveLock to refresh dependencies)
  • WHOSUSING.md
  • Other (please describe):

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.

Changes in this PR

Fixes a race condition where a COMPLETED asyncComplete event task could move back to IN_PROGRESS if task update is received before task creation occurs.

@apanicker-nflx apanicker-nflx added the type: bug bugs/ bug fixes label Aug 18, 2022
@apanicker-nflx apanicker-nflx force-pushed the execute_logic_for_event branch from a5517f4 to c96e8c6 Compare August 18, 2022 00:58
@apanicker-nflx apanicker-nflx merged commit 8b62e35 into main Aug 18, 2022
@apanicker-nflx apanicker-nflx deleted the execute_logic_for_event branch August 18, 2022 21:54
manan164 added a commit to manan164/conductor that referenced this pull request Nov 2, 2022
pmchung pushed a commit to routific/conductor that referenced this pull request Sep 6, 2023
…flix#3187)

* event system task implementation is split between start and execute

* added integ test
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug bugs/ bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants