wenjin272 commented on issue #829: URL: https://github.com/apache/flink-agents/issues/829#issuecomment-4890070845
Hi @weiqingy, thank you for the detailed breakdown. I recommend deprecating `local_runner`, `local_execution_environment`, and their associated components (such as documentation and `LocalMemoryObject`) immediately, while we are still in beta. Initially, we implemented `local_execution_environment` primarily to allow users to perform simple validation of Agent logic in environments without a JVM. However, since Flink Agents are designed for users who intend to run them within Flink, using Flink’s MiniCluster is likely a better validation approach. Moreover, maintaining `local_execution_environment` incurs significant overhead. We can migrate the tests based on `local_execution_environment` to the embedded minicluster and remove the relevant descriptions from the documentation. -- 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]
