xintongsong commented on a change in pull request #11284: 
[FLINK-15911][runtime] Make Flink work with NAT.
URL: https://github.com/apache/flink/pull/11284#discussion_r395429730
 
 

 ##########
 File path: 
flink-core/src/main/java/org/apache/flink/configuration/TaskManagerOptions.java
 ##########
 @@ -94,19 +94,28 @@
                                " or if it has been quarantined by another 
actor system.");
 
        /**
-        * The config parameter defining the task manager's hostname.
+        * The external address of the network interface where the TaskManager 
is exposed.
         * Overrides {@link #HOST_BIND_POLICY} automatic address binding.
         */
        @Documentation.Section({Documentation.Sections.COMMON_HOST_PORT, 
Documentation.Sections.ALL_TASK_MANAGER})
        public static final ConfigOption<String> HOST =
                key("taskmanager.host")
                        .stringType()
                        .noDefaultValue()
-                       .withDescription("The address of the network interface 
that the TaskManager binds to." +
-                               " This option can be used to define explicitly 
a binding address. Because" +
-                               " different TaskManagers need different values 
for this option, usually it is specified in an" +
+                       .withDescription("The external address of the network 
interface where the TaskManager is exposed." +
+                               " Because different TaskManagers need different 
values for this option, usually it is specified in an" +
                                " additional non-shared TaskManager-specific 
config file.");
 
+       /**
+        * The local address of the network interface that the task manager 
binds to.
+        */
+       public static final ConfigOption<String> BIND_HOST =
+               key("taskmanager.bind-host")
 
 Review comment:
   Same here. TM address and bind-address are shared by RPC service, netty 
shuffle service and queryable kv state.
   
   I'm actually thinking about changing "jobmanager.rpc.address" to 
"jobmanager.host" as a follow up, to make the config keys consistently using 
"host".

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to