aglinxinyuan opened a new pull request, #6305: URL: https://github.com/apache/texera/pull/6305
### What changes were proposed in this PR? Removes the dead `handleActorCommand` method from `WorkflowWorker`. It is a `Receive` whose body is a bare `println(c)`, it is not part of the actor's `receive` chain (`receive = super.receive orElse handleDirectInvocation orElse handleTriggerClosure`), and it has no callers — the live `handleActorCommand` is a separate method on `DPThread`. Deleting it removes a debug leftover and raises coverage by removal rather than cementing dead code with a test. ### Any related issues, documentation, discussions? Closes #6304. Follow-up to a review comment by @mengw15 on #6289. ### How was this PR tested? No new tests — this only deletes unreachable code. Verified locally (Java 17): the amber module compiles, `WorkerSpec` (which exercises `WorkflowWorker`) passes, and `scalafixAll --check` is clean (the `ActorCommand` import remains used via `ActorCommandElement`). ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.8 [1M context]) -- 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]
