carloea2 opened a new pull request, #8236: URL: https://github.com/apache/texera/pull/8236
### What is this PR for? Add the worker's `EvaluatedValue` reply to the generic `ControlReturn` transport. This prevents evaluated Python expression results from being replaced by an empty return before they reach the coordinator. Closes #8235 ### What type of PR is it? Bug Fix ### How was this PR tested? ```text python -c "import sys,pytest; sys.path[:0]=[worktree amber source]; raise SystemExit(pytest.main(['amber/src/test/python/core/architecture/rpc/test_async_rpc_handler_initializer.py','amber/src/test/python/core/architecture/rpc/test_async_rpc_server.py','amber/src/test/python/core/architecture/handlers/control/test_evaluate_expression_handler.py','-q','-p','no:cacheprovider']))" 81 passed ruff check amber/src/main/python amber/src/test/python All checks passed ruff format --check amber/src/main/python amber/src/test/python 213 files already formatted ``` The Python protobuf bindings were regenerated from the changed descriptor for verification and remain ignored as required by the repository. A direct post-fix transport probe produced: ```text packed_type=EvaluatedValue packed_value=42 round_trip=True ``` Generated-by: Codex -- 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]
