Hexiaoqiao commented on PR #5131:
URL: https://github.com/apache/hadoop/pull/5131#issuecomment-1317074076

   Thanks @slfan1989 , Great work here! 
   One concern out of this PR, we have HDFS RBF / YARN Federation / KMS both 
depend on zookeeper to store DelegationToken. From my view, both of their logic 
are similar, we should consider implement one common module about 
ZKDelegationTokenManager? Thus it will reduce overhead to implement at each 
sub-project side. WDYT? cc @goiri 
   NOTE: This is not blocker comments. Thanks.


-- 
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: common-issues-unsubscr...@hadoop.apache.org

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

Reply via email to