GitHub user Yicong-Huang closed the discussion with a comment: Revisit the 
early decision regarding "Remove unused RPCs and disable reconfiguration"

Follow-up: PR #4220 re-enabled the engine-side reconfiguration handlers, but 
the web-service entrypoint 
`ExecutionReconfigurationService.performReconfigurationOnResume` was still in 
its `throw new RuntimeException("reconfiguration is tentatively disabled.")` 
state (the body that actually issues the controller RPC was commented out back 
in #2970 and was not restored alongside #4220). Result: pause → modify operator 
property → resume kills the workflow with that exception even though the engine 
path is now ready.

PR #4531 wires the entrypoint back to 
`controllerInterface.reconfigureWorkflow`, adapted to the current 
`UpdateExecutorRequest(targetOpId, newExecInitInfo)` proto shape (no more 
proto-Any boxing of `OpExecInitInfo`). One scope cut: `StateTransferFunc` is no 
longer threaded through, since the new `UpdateExecutorRequest` schema does not 
carry it — flagged in the PR description in case anyone wants to bring it back.

GitHub link: 
https://github.com/apache/texera/discussions/4016#discussioncomment-16720879

----
This is an automatically sent email for [email protected].
To unsubscribe, please send an email to: [email protected]

Reply via email to