New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
YARN-11399 Make DelegationTokenRenwer timeout and retry feature configurable #5234
base: trunk
Are you sure you want to change the base?
Conversation
@Daniel-009497 Thank you very much for your contribution. From my personal point of view, this parameter should not be set to false. If it is set to false, RM will not be able to renew token. I think this change may be unreasonable. |
💔 -1 overall
This message was automatically generated. |
@slfan1989 Noy really, if the delegation token renewer pool tracker feature is disabled, the previous token renew logic still take effect. This new feature is just a enhancement feature based on currently exsiting token renew feature. |
@Daniel-009497 I understand your explanation, it seems reasonable, I will check the current logic later.
|
@brahmareddybattula @ayushtkn @bibinchundatt @hemanthboyina Could you pls help to review and merge? |
@Daniel-009497 Thank you very much for your contribution, I read this part of code and I think this variable( This optimization comes from JIRA: YARN-9768. RM Renew Delegation token thread should timeout and retry.
If this variable is set to We can read YARN-9768 carefully. The author at that time should have set this variable for compatibility with previous unit tests. We can see the following code. org.apache.hadoop.yarn.server.resourcemanager.security#DelegationTokenRenewer L939
Let's wait for other partners' suggestions. |
ResourceManager Delegation token renew timeout and retry mechanism is introduced in Hadoop 3.3.1, The switch of this feature is already there, but it is true and can not be disabled.
So this patch introduce a configuration to make this feature configurable.