[ 
https://issues.apache.org/jira/browse/YARN-8028?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16403154#comment-16403154
 ] 

Wangda Tan commented on YARN-8028:
----------------------------------

Thanks [~bibinchundatt] for review.

For BadRequestException/ForbiddenException, I would prefer to change to use 
status code for consistency. 
{quote} # I think we shouldnt directly log the params inputs this could cause 
*log forging*{quote}
Done.

For #2/#3, I would prefer make minimum changes in this patch only, let's move 
improvement discussion to other JIRAs you mentioned. Existing JIRA is targeted 
to get ACL for single queue, which should be relatively efficiency comparing to 
get ACLs of all queues.

> Support authorizeUserAccessToQueue in RMWebServices
> ---------------------------------------------------
>
>                 Key: YARN-8028
>                 URL: https://issues.apache.org/jira/browse/YARN-8028
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Wangda Tan
>            Assignee: Wangda Tan
>            Priority: Major
>         Attachments: YARN-8028.001.patch, YARN-8028.002.patch
>
>
> Currently we have {{QueueUserACLInfo}} in ApplicationClient, we should 
> support similar API in REST API.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to