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


Reply via email to