Skip to content

fix(engine): keep notification streams open across task result states - #89

Merged
Zhoujie628 merged 1 commit into
project-openan:mainfrom
Zhoujie628:main
Sep 30, 2026
Merged

Zhoujie628 merged 1 commit into
project-openan:mainfrom
Zhoujie628:main

Conversation

@Zhoujie628

Copy link
Copy Markdown
Contributor

Summary

  • Keep Notification-T subscriptions independent from individual task result states.
  • Preserve terminal-event completion for one-shot Task-T streams.
  • Cover terminal notification events, subsequent heartbeats, and Task-T completion in tests.

Validation

  • mvn -q -pl workflow-engine -am spotless:check test -DskipITs (passed)
  • git diff --cached --check (passed before commit).

For maintainer review; do not auto-merge.

Signed-off-by: Zhoujie628 <zhoujie628@users.noreply.github.com>
@Zhoujie628
Zhoujie628 requested a review from a team as a code owner September 26, 2026 11:14
@Zhoujie628
Zhoujie628 merged commit 61fae8c into project-openan:main Sep 30, 2026
3 checks passed
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