9uapaw commented on a change in pull request #4060:
URL: https://github.com/apache/hadoop/pull/4060#discussion_r831268938
##########
File path:
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/NodeLabel.md
##########
@@ -170,6 +170,14 @@ Applications can use following Java APIs to specify node
label to request
* `ResourceRequest.setNodeLabelExpression(..)` to set node label expression
for individual resource requests. This can overwrite node label expression set
in ApplicationSubmissionContext
* Specify `setAMContainerResourceRequest.setNodeLabelExpression` in
`ApplicationSubmissionContext` to indicate expected node label for application
master container.
+__Default AM node-label Configuration__
+
+Property | Value
+----- | ------
+yarn.resourcemanager.node-labels.am.default-node-label-expression | Besides
when ApplicationMaster of application is not specified any node-labels, then
the configuration will attach the default node-label to AM. The default of this
config is disabled.
+
Review comment:
Could you rephrase it to:
"Overwrites default-node-label-expression only for the ApplicationMaster
container. It is disabled by default."
--
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: common-issues-unsubscr...@hadoop.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
