carloea2 opened a new pull request, #8166: URL: https://github.com/apache/texera/pull/8166
### What changes were proposed in this PR? Guard workflow logic changes when the workflow exists but its execution service has not been initialized. The handler now returns a clear workflow state error instead of leaking a null pointer error. A regression test covers the missing execution state while retaining the existing successful logic change and missing workflow cases. ### Any related issues, documentation, discussions? Closes #8165 ### How was this PR tested? ```shell sbt -no-colors "WorkflowExecutionService/testOnly org.apache.texera.web.resource.WorkflowWebsocketResourceSpec" sbt -no-colors scalafmtCheckAll "scalafixAll --check" sbt -no-colors "WorkflowExecutionService/dist" ``` The focused suite passed 16 tests. For the live check, I started the packaged computing unit service on localhost, connected a workflow WebSocket before execution initialization, and sent a `ModifyLogicRequest`. Before: `WorkflowErrorEvent` contained a null pointer error because `executionService` was null. After: `WorkflowErrorEvent` contains `workflow execution is not initialized`. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Codex GPT-5 -- 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]
