spoon-lz commented on a change in pull request #18119: URL: https://github.com/apache/flink/pull/18119#discussion_r776950342
########## File path: flink-kubernetes/src/main/java/org/apache/flink/kubernetes/utils/KubernetesUtils.java ########## @@ -113,7 +156,7 @@ public static void checkAndUpdatePortConfigOption( * @param port port config option * @return valid port */ - public static Integer parsePort(Configuration flinkConfig, ConfigOption<String> port) { + public static Integer parseFixedPort(Configuration flinkConfig, ConfigOption<String> port) { Review comment: I changed the name because I added the `updateDynamicPorts` method to make it easier to distinguish. Now it can be restored -- 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