[ 
https://issues.apache.org/jira/browse/HADOOP-6510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12830389#action_12830389
 ] 

Kan Zhang commented on HADOOP-6510:
-----------------------------------

patch looks good. Some nits.

bq. JavaDoc for some public classes and public methods are missing, e.g., 
ProxyUsers. createProxyUsers()'s JavaDoc doesn't match its signature. I wonder 
why test-patch didn't catch it.

bq. In ConnectionHeader.write(), you shouldn't try to search the return value 
of ugi.getTokens(), but use the token object associate with the connection.


> doAs for proxy user
> -------------------
>
>                 Key: HADOOP-6510
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6510
>             Project: Hadoop Common
>          Issue Type: New Feature
>            Reporter: Jitendra Nath Pandey
>            Assignee: Jitendra Nath Pandey
>         Attachments: HADOOP-6510.10.patch, HADOOP-6510.11.patch, 
> HADOOP-6510.12.patch, HADOOP-6510.14.patch, HADOOP-6510.15.patch, 
> HADOOP-6510.16.patch, HADOOP-6510.17.patch, HADOOP-6510.18.patch, 
> HADOOP-6510.19.patch, HADOOP-6510.20.patch, HADOOP-6510.8.patch
>
>
> This jira will add support for a superuser authenticating on behalf of a 
> proxy user.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to