hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] xiaoyuyao commented on a change in pull request #526: HDDS-1183. Override getDelegationToken API for OzoneFileSystem. Contr…
Date Fri, 01 Mar 2019 19:05:20 GMT
xiaoyuyao commented on a change in pull request #526: HDDS-1183. Override getDelegationToken
API for OzoneFileSystem. Contr…
URL: https://github.com/apache/hadoop/pull/526#discussion_r261722027
 
 

 ##########
 File path: hadoop-ozone/ozonefs/src/main/java/org/apache/hadoop/fs/ozone/OzoneFileSystem.java
 ##########
 @@ -669,6 +676,12 @@ public Path getWorkingDirectory() {
     return workingDir;
   }
 
+  @Override
+  public Token<?> getDelegationToken(String renewer) throws IOException {
+    return securityEnabled? adapter.getDelegationToken(renewer) :
+        super.getDelegationToken(renewer);
 
 Review comment:
   bq. "should fetch DT from both", that is handled by addDelegationTokens() in the TokenIssuer.
   Here we follow the FileSystem contract to return om delegation token if ozone security
is enabled otherwise null as speced in FileSystem#getDelegationToken().

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org


Mime
View raw message