[jira] [Commented] (HADOOP-10566) Refactor proxyservers out of ProxyUsers

["Hadoop QA (JIRA)" <ji...@apache.org>]

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

Hadoop QA commented on HADOOP-10566:
------------------------------------

{color:red}-1 overall{color}.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12643002/HADOOP-10566.patch
  against trunk revision .

    {color:green}+1 @author{color}.  The patch does not contain any @author tags.

    {color:green}+1 tests included{color}.  The patch appears to include 3 new or modified test files.

    {color:red}-1 javac{color:red}.  The patch appears to cause the build to fail.

Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/3894//console

This message is automatically generated.

> Refactor proxyservers out of ProxyUsers
> ---------------------------------------
>
>                 Key: HADOOP-10566
>                 URL: https://issues.apache.org/jira/browse/HADOOP-10566
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: security
>            Reporter: Benoy Antony
>            Assignee: Benoy Antony
>         Attachments: HADOOP-10566.patch
>
>
> HADOOP-10498 added proxyservers feature in ProxyUsers. It is beneficial to treat this as a separate feature since 
> 1> The ProxyUsers is per proxyuser where as proxyservers is per cluster. The cardinality is different. 
> 2> The ProxyUsers.authorize() and ProxyUsers.isproxyUser() are synchronized and hence share the same lock  and impacts performance.
> Since these are two separate features, it will be an improvement to keep them separate. It also enables one to fine-tune each feature independently.



--
This message was sent by Atlassian JIRA
(v6.2#6252)