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

Chris Nauroth commented on HADOOP-13422:
----------------------------------------

{code}
        private final javax.security.auth.login.Configuration baseConfig = 
javax.security.auth.login.Configuration
                .getConfiguration();
{code}

I expect pre-commit will flag a nitpick about indentation and line length 
exceeding 80 characters here, so we'll need one more patch revision.

I'm in favor of the approach though.  This will help avoid some bugs until we 
can implement a long-term fix that makes use of ZOOKEEPER-2139.  There is 
already similar working code in Hive.  (See the {{LlapZookeeperRegistryImpl}} 
class.)  I know Sergey was able to demonstrate that this fix works through 
manual testing.

[~asuresh], are you interested in reviewing this?  I'll give it some time 
before I consider committing.

> ZKDelegationTokenSecretManager JaasConfig does not work well with other ZK 
> users in process
> -------------------------------------------------------------------------------------------
>
>                 Key: HADOOP-13422
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13422
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: Sergey Shelukhin
>            Assignee: Sergey Shelukhin
>         Attachments: HADOOP-13422.patch
>
>
> There's a race in the globals. The non-global APIs from ZOOKEEPER-2139  are 
> not available yet in a stable ZK version and there's no timeline for 
> availability, so for now it would help to make SM aware of other users of the 
> global config.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to