venkata91 commented on code in PR #22509:
URL: https://github.com/apache/flink/pull/22509#discussion_r1186738477


##########
docs/layouts/shortcodes/generated/yarn_config_configuration.html:
##########
@@ -182,5 +182,15 @@
             <td>String</td>
             <td>Specify YARN node label for the Flink TaskManagers, it will 
override the yarn.application.node-label for TaskManagers if both are set.</td>
         </tr>
+        <tr>
+          <td><h5>yarn.view.acls</h5></td>

Review Comment:
   Should we link to either this 
[doc](https://docs.cloudera.com/documentation/enterprise/latest/topics/cm_mc_yarn_acl.html)
 or this 
[doc](https://docs.cloudera.com/runtime/7.2.7/yarn-security/yarn-security.pdf)? 
Looks like there are more ACL rules and syntax. For eg:
   1. Group names are case sensitive.
   2. If an ACL starts with a single space, then it must consist of groups only.
   



-- 
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