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

Daryn Sharp commented on HDFS-3113:
-----------------------------------

This is a big patch, so here's what I've immediately noticed:

The {{uri}} should be constructed as 
{{URI.create(uri.getScheme()+"://"+uri.getAuthority())}} in case it doesn't 
include the default port.  Otherwise the current patch I think will cause -1 to 
be in the uri.

The {{httpFSAddr}} (used for the token service) needs to be based on 
{{getCanonicalUri}} which adds the default port if necessary.  Similarly, 
{{getDefaultPort}} should be implemented.

A {{TokenRenewer}} is needed that is registered for the service loader.


                
> httpfs does not support delegation tokens
> -----------------------------------------
>
>                 Key: HDFS-3113
>                 URL: https://issues.apache.org/jira/browse/HDFS-3113
>             Project: Hadoop HDFS
>          Issue Type: New Feature
>    Affects Versions: 0.23.3
>            Reporter: Alejandro Abdelnur
>            Assignee: Alejandro Abdelnur
>             Fix For: 0.24.0, 0.23.3
>
>         Attachments: HDFS-3113.patch, HDFS-3113.patch, HDFS-3113.patch
>
>
> httpfs does not support calls to get/renew tokens nor delegation token 
> authentication.

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