krasinski edited a comment on pull request #12462: URL: https://github.com/apache/flink/pull/12462#issuecomment-638494158
thank you for the review @walterddr, I've addressed all the comments and added some more unit tests I've also introduced a small change in `hasHDFSDelegationToken` method - taking advantage of the UGI parameter instead of retrieving it one more time seems more natural edit: need to find another way to init kerberos config, tests worked on CI but failed on CI ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
