gaborgsomogyi commented on a change in pull request #18664:
URL: https://github.com/apache/flink/pull/18664#discussion_r840297210
##########
File path:
flink-yarn/src/main/java/org/apache/flink/yarn/YarnClusterDescriptor.java
##########
@@ -1090,9 +1095,17 @@ private ApplicationReport startAppMaster(
final ContainerLaunchContext amContainer =
setupApplicationMasterContainer(yarnClusterEntrypoint,
hasKrb5, processSpec);
- // setup security tokens
+ // Here we have the new and the old delegation token framework
together temporarily.
+ // The reason is simple, adding a horror complicated several thousand
lines PR is not a good
+ // idea so in the upcoming commits will remove the old DT framework
and only the
+ // new will be in place.
+
Review comment:
OK, removed. It's not required just wanted to give some context for
reviewers.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
