Yicong-Huang commented on PR #4515: URL: https://github.com/apache/texera/pull/4515#issuecomment-4321081602
Closing — the test motivation for this change can be solved without a new event broadcast: `Await.result(resumeWorkflow)` already blocks until all workers have transitioned to RUNNING (the resumeWorkflow Future completes inside ResumeHandler's `.map` block, after every worker future has resolved with its updated state). Adding a duplicate ExecutionStateUpdate(RUNNING) emit would only mean an extra round-trip on every resume, with no behavior change for existing consumers (web layer already sets RUNNING via `.onSuccess`). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
