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

Konstantin Boudnik commented on HADOOP-6944:
--------------------------------------------

Yup, I think that's it. 

One nit:
 - would it be better to throw NoSuchMethodException from the body of default 
{{writeToFile}} implementation here
{noformat}
+    ProxyUserDefinitions pud = new ProxyUserDefinitions() {
+      public boolean writeToFile(URI filePath) throws IOException {
+        return false;
+      };
{noformat}

Other than that it looks good.

> [Herriot] Implement a functionality for getting proxy users definitions like 
> groups and hosts.
> ----------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-6944
>                 URL: https://issues.apache.org/jira/browse/HADOOP-6944
>             Project: Hadoop Common
>          Issue Type: Task
>          Components: test
>            Reporter: Vinay Kumar Thota
>            Assignee: Vinay Kumar Thota
>         Attachments: HADOOP-6944.patch, HADOOP-6944.patch, HADOOP-6944.patch
>
>
> Gridmix should require a proxy user's file for impersonating various jobs. 
> So, implement couple of methods for getting the proxy users list and a proxy 
> users file (it's a combination of proxy users and groups) based on cluster 
> configuration.
> The proxy users list should require for map reduce jobs and proxy users file 
> should require for gridmix jobs.
> The following are methods signature,
> public ProxyUserDefinitions getHadoopProxyUsers() - get the list of proxy 
> users list based on cluster configuration.

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