rosemarYuan commented on code in PR #821: URL: https://github.com/apache/flink-agents/pull/821#discussion_r3702749847
########## python/flink_agents/runtime/remote_execution_environment.py: ########## @@ -45,6 +45,8 @@ _CONFIG_FILE_NAME = "config.yaml" _LEGACY_CONFIG_FILE_NAME = "flink-conf.yaml" +_AGENT_PLAN_PREFLIGHT_CLASS = "org.apache.flink.agents.plan.AgentPlanPreflight" +_AGENT_PLAN_PREFLIGHT_METHOD = "findValidationError" Review Comment: Thanks for the review. I’ve addressed the comments about YAML validation, validated plan serialization, event IDs, and the preflight validator naming/contract. For the Log4j dependency scope and packaging changes in the POM files, I reverted them from this PR because they are unrelated to CEL trigger conditions. I opened [#954](https://github.com/apache/flink-agents/pull/954) to handle that issue separately, including the required packaging and deployment verification. -- 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]
