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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
