davidradl commented on code in PR #975: URL: https://github.com/apache/flink-kubernetes-operator/pull/975#discussion_r2065811570
########## helm/flink-kubernetes-operator/conf/flink-conf.yaml: ########## @@ -24,8 +24,8 @@ parallelism.default: 1 # These should be kept in-sync with the flink dist env.java.opts.all defaults (for the given flink version) in: flink-dist/src/main/resources/config.yaml # Flink 1.18 uses env.java.opts.all, if a user supplies their own version of these opts in their FlinkDeployment the options below will be overridden. # env.java.default-opts.all is used for 1.19 onwards so users can supply their own opts.all in their Job deployments and have these appended. Review Comment: I see in the comment above that the below content should be kept in sync with `flink-dist/src/main/resources/config.yaml.` I suggest that we amend the comment - as it will no longer be true. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
