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

Sanjay Radia commented on HADOOP-7967:
--------------------------------------

+1 - i have some changes in the delta patch I am attaching.
* Some minor improvements to javadoc and added comments to some tests (in my 
delta patch)
* Given that Hadoop 1 did not have audience annotations we can't make 
getDelegationToken protected. I have fixed that in my delta patch. Note the 
changes you made in tests to call addDelegationTokens instead of 
getDelegationToken are correct and should remain.
* File a Jira to make getDelegationTokens protected - lets see if community 
feels this can be done at some stage.
* File a Jira to make the corresponding changes to 
FileContext/AbstractFileSystem as we discussed.

Thanks for adding more tests and for refactoring some of the test internals.

                
> Need generalized multi-token filesystem support
> -----------------------------------------------
>
>                 Key: HADOOP-7967
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7967
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: fs, security
>    Affects Versions: 0.23.1, 0.24.0
>            Reporter: Daryn Sharp
>            Assignee: Daryn Sharp
>            Priority: Critical
>         Attachments: HADOOP-7967-2.patch, HADOOP-7967-3.patch, 
> HADOOP-7967-4.patch, HADOOP-7967-compat.patch, HADOOP-7967.newapi.2.patch, 
> HADOOP-7967.newapi.3.patch, HADOOP-7967.newapi.patch, HADOOP-7967.patch, 
> hadoop7967-javadoc.patch
>
>
> Multi-token filesystem support and its interactions with the MR 
> {{TokenCache}} is problematic.  The {{TokenCache}} tries to assume it has the 
> knowledge to know if the tokens for a filesystem are available, which it 
> can't possibly know for multi-token filesystems.  Filtered filesystems are 
> also problematic, such as har on viewfs.  When mergeFs is implemented, it too 
> will become a problem with the current implementation.  Currently 
> {{FileSystem}} will leak tokens even when some tokens are already present.
> The decision for token acquisition, and which tokens, should be pushed all 
> the way down into the {{FileSystem}} level.  The {{TokenCache}} should be 
> ignorant and simply request tokens from each {{FileSystem}}.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to