wanglijie95 commented on code in PR #20056:
URL: https://github.com/apache/flink/pull/20056#discussion_r910568148
##########
flink-core/src/main/java/org/apache/flink/configuration/TaskManagerOptionsInternal.java:
##########
@@ -32,4 +32,15 @@ public class TaskManagerOptionsInternal {
.noDefaultValue()
.withDescription(
"**DO NOT USE** The metadata of TaskManager's
ResourceID to be used for logging.");
+
+ /**
+ * The ID of the node where the TaskManager is located. In Yarn and Native
Kubernetes mode, this
+ * option will be set by resource manager when launch a container for the
task executor. In
+ * other modes, this option will not be set. This option is only used
internally.
+ */
+ public static final ConfigOption<String> TASK_MANAGER_NODE_ID =
+ key("internal.taskmanager.node-id")
+ .stringType()
+ .noDefaultValue()
+ .withDescription("ID of the node where the TaskManager is
located.");
Review Comment:
Sure. I perfer to use `on`. I will fix it in all places.
--
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
