weiqingy commented on PR #722: URL: https://github.com/apache/flink-agents/pull/722#issuecomment-4667144105
Thanks for the review, and agreed on the structured-validation direction. On `local_runner` / `local_execution_environment` — good point. They exist as a pure-Python, in-process path (selected when no Flink `StreamExecutionEnvironment` is passed), so they're not the production path, yet every execution-layer change has to be carried through them too — as this PR's tool-invocation assertions were. I've opened #829 to scope what deprecating them would involve (migration path for the e2e tests and quickstart/example scripts that depend on the local path, and a deprecation timeline) so we can weigh it for a future version. -- 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]
