reswqa commented on code in PR #3090:
URL: https://github.com/apache/amoro/pull/3090#discussion_r1714605948
##########
amoro-ams/amoro-ams-server/src/main/java/org/apache/amoro/server/manager/FlinkOptimizerContainer.java:
##########
@@ -80,25 +84,15 @@ public class FlinkOptimizerContainer extends
AbstractResourceContainer {
public static final String FLINK_HOME_PROPERTY = "flink-home";
public static final String FLINK_CONFIG_PATH = "/conf";
public static final String FLINK_CONFIG_YAML = "/flink-conf.yaml";
+ // flink version >= 1.20 use it first
+ public static final String NEW_FLINK_CONFIG_YAML = "/config.yaml";
Review Comment:
> OLD_FLINK_CONFIG_YAML="/flink-conf.yaml"
I suggest using `LEGACY_FLINK_CONF_YAML`.
--
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]
