kunwp1 commented on code in PR #5565:
URL: https://github.com/apache/texera/pull/5565#discussion_r3379336787
##########
frontend/src/app/workspace/service/execute-workflow/execute-workflow.service.ts:
##########
@@ -354,6 +354,15 @@ export class ExecuteWorkflowService {
};
}
+ /**
+ * Reset execution status and worker assignments. Unlike
resetExecutionState(),
+ * which resets only the status, this also clears the worker assignments.
+ */
+ public resetExecutionAndWorkers(): void {
+ this.resetExecutionState();
+ this.assignedWorkerIds.clear();
+ }
Review Comment:
Done. `resetExecutionAndWorkers()` now goes through `updateExecutionState({
state: Uninitialized })` instead of `resetExecutionState()`, so the reset is
broadcast on `executionStateStream` and `MenuComponent`/`ResultPanelComponent`
drop the previous unit's status. The spec now asserts the stream emits.
--
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]
