wenjin272 commented on PR #1347: URL: https://github.com/apache/flink-agents/pull/1347#issuecomment-6734343306
Thanks for the contribution! #1337 has now been merged. Could you align the Python behavior in this PR as discussed earlier? - Reuse `flink_version_manager`: Flink 1.20 should prefer `flink-conf.yaml`, falling back to `config.yaml`; Flink 2.x should only read `config.yaml`. - Use a lightweight Python legacy parser for `flink-conf.yaml` and standard YAML parsing for `config.yaml`, with tests covering both behaviors. -- 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]
