weiqingy opened a new pull request, #882: URL: https://github.com/apache/flink-agents/pull/882
Linked issue: #829 ### Purpose of change Second of two PRs retiring the Python in-process local execution path, per #829. This PR removes the local path itself. Stacked on #881 (the additive test backfill). Until #881 merges, this PR's diff includes #881's commits; the net-new change here is the removal, in the last two commits — "Remove local-path tests and migrate remaining tests off it" and "Delete the local execution path and require a Flink environment". I'll rebase once #881 lands. What this removes: - `local_runner.py` and `local_execution_environment.py` — the pure-Python, single-process execution path selected when no Flink environment was passed. - The execution-environment `from_list` / `to_list` test-only API and the local-only `get_tool_request_events` hook. - The local-path tests, now covered by the remote (MiniCluster) tests added in #881; and the local-path sections of the documentation. `get_execution_environment` now requires a `StreamExecutionEnvironment` and raises a clear error when none is provided, instead of falling back to the local path. The RAG and quickstart examples are migrated to the remote DataStream pattern. `LocalMemoryObject` is kept as an in-memory test fixture (it is independent of the runner and still used by unit tests). ### Tests The Python unit lane and the remote end-to-end suite pass. The local-path tests are removed because equivalent behavior is now exercised on a real Flink MiniCluster by the tests added in #881 (durable-execute sync-exception and kwargs, built-in action content and `get_resource`, MCP tool and prompt invocation, workflow short-term-memory semantics, and the apply-by-name guard). ### API Yes — this removes public surface: the execution-environment `from_list` / `to_list` methods, `get_tool_request_events`, and the no-argument `get_execution_environment()` form (a Flink environment is now required). This is a deliberate removal of the non-production local path while the project is in beta. ### Documentation - [ ] `doc-needed` - [ ] `doc-not-needed` - [x] `doc-included` -- 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]
