[
https://issues.apache.org/jira/browse/HADOOP-10566?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13995284#comment-13995284
]
Benoy Antony commented on HADOOP-10566:
---------------------------------------
Doing a resend since the mail servers were down last week:
[~wheat9], [~arpitagarwal] , Could one of you please review and commit this
patch ?
This is already reviewed by [~daryn]. His comments are addressed. It already
passed Jenkins.
The earlier comments related to commits seem to be some error. Looks like, it
was due to confusion with HADOOP-10556.
> 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
> Affects Versions: 2.4.0
> Reporter: Benoy Antony
> Assignee: Benoy Antony
> Attachments: HADOOP-10566.patch, HADOOP-10566.patch,
> HADOOP-10566.patch, HADOOP-10566.patch, 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)
