[jira] [Commented] (YARN-8455) Add basic acl check for all TS v2 REST APIs

[ https://issues.apache.org/jira/browse/YARN-8455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16527953#comment-16527953 ] Sunil Govindan commented on YARN-8455: -- Thanks [~rohithsharma] for testing and verifying the same.

[jira] [Commented] (YARN-8455) Add basic acl check for all TS v2 REST APIs

[ https://issues.apache.org/jira/browse/YARN-8455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16527231#comment-16527231 ] genericqa commented on YARN-8455: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-8455) Add basic acl check for all TS v2 REST APIs

[ https://issues.apache.org/jira/browse/YARN-8455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16527047#comment-16527047 ] genericqa commented on YARN-8455: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-8455) Add basic acl check for all TS v2 REST APIs

[ https://issues.apache.org/jira/browse/YARN-8455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16527000#comment-16527000 ] Rohith Sharma K S commented on YARN-8455: - While testing I found one of the REST query was not

[jira] [Commented] (YARN-8455) Add basic acl check for all TS v2 REST APIs

[ https://issues.apache.org/jira/browse/YARN-8455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16526586#comment-16526586 ] Sunil Govindan commented on YARN-8455: -- Latest patch looks good to me. +1 Committing shortly. > Add

[jira] [Commented] (YARN-8455) Add basic acl check for all TS v2 REST APIs

[ https://issues.apache.org/jira/browse/YARN-8455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16525678#comment-16525678 ] genericqa commented on YARN-8455: - | (/) *{color:green}+1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-8455) Add basic acl check for all TS v2 REST APIs

[ https://issues.apache.org/jira/browse/YARN-8455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16525622#comment-16525622 ] Rohith Sharma K S commented on YARN-8455: - Thanks [~sunilg] for the review. Attached patch along

[jira] [Commented] (YARN-8455) Add basic acl check for all TS v2 REST APIs

[ https://issues.apache.org/jira/browse/YARN-8455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16523982#comment-16523982 ] Sunil Govindan commented on YARN-8455: -- I think this is the first step towards reaching to a complete

[jira] [Commented] (YARN-8455) Add basic acl check for all TS v2 REST APIs

[ https://issues.apache.org/jira/browse/YARN-8455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16523159#comment-16523159 ] Rohith Sharma K S commented on YARN-8455: - bq. Can we also sync the exception handling similar to

[jira] [Commented] (YARN-8455) Add basic acl check for all TS v2 REST APIs

[ https://issues.apache.org/jira/browse/YARN-8455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16523128#comment-16523128 ] Bibin A Chundatt commented on YARN-8455: [~rohithsharma] Can we also sync the exception handling

[jira] [Commented] (YARN-8455) Add basic acl check for all TS v2 REST APIs

[ https://issues.apache.org/jira/browse/YARN-8455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16523006#comment-16523006 ] genericqa commented on YARN-8455: - | (x) *{color:red}-1 overall{color}* | \\ \\ || Vote || Subsystem ||

[jira] [Commented] (YARN-8455) Add basic acl check for all TS v2 REST APIs

[ https://issues.apache.org/jira/browse/YARN-8455?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16522958#comment-16522958 ] Rohith Sharma K S commented on YARN-8455: - cc:/ [~sunilg] > Add basic acl check for all TS v2