umamaheswararao edited a comment on pull request #2260: URL: https://github.com/apache/hadoop/pull/2260#issuecomment-687277527
Thanks @steveloughran for review! In addition to his comments, I have the following question/comment. In practice, I am thinking this lazy initialization will not give us real benefit in the following case with MR/YARN. In DelegationTokenIssuer#collectDelegationTokens, it will try to get tokens from all children. ``` // Now collect the tokens from the children. final DelegationTokenIssuer[] ancillary = issuer.getAdditionalTokenIssuers(); if (ancillary != null) { for (DelegationTokenIssuer subIssuer : ancillary) { collectDelegationTokens(subIssuer, renewer, credentials, tokens); } } ``` I am wondering this call will make all fs to be initialized. That means we are not getting this lazy initialization benefit fully. Did you consider this case? Is there a way to avoid this ? MR/YARN team can suggest something if we have alternative way to load tokens. CC: @wangdatan @sunilgovind @rohithsharmaks CC: @xiaoyuyao ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org