gaborgsomogyi commented on a change in pull request #18664:
URL: https://github.com/apache/flink/pull/18664#discussion_r802892543
##########
File path:
flink-yarn/src/main/java/org/apache/flink/yarn/YarnClusterDescriptor.java
##########
@@ -1245,6 +1258,21 @@ private ApplicationReport startAppMaster(
return report;
}
+ private void setTokensFor(ContainerLaunchContext containerLaunchContext)
throws IOException {
+ LOG.info("Adding delegation tokens to the AM container.");
+
+ Credentials credentials =
UserGroupInformation.getCurrentUser().getCredentials();
+
+ DelegationTokenManager delegationTokenManager =
+ new DelegationTokenManager(flinkConfiguration);
+ delegationTokenManager.obtainDelegationTokens(credentials);
Review comment:
`obtainDelegationTokens` obtains tokens but these are mustn't be
invalidated if you mean that.
`start/stop` just starts and stops automatic re-obtain and propagation. Not
yet see why stopping this needs to be called close. In this case `close` would
be no-op because manual obtain is used.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
