-
Notifications
You must be signed in to change notification settings - Fork 9.1k
HADOOP-17578. Improve UGI debug log to help troubleshooting TokenCach… #2762
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @xiaoyuyao for the work.
Overall looks great, I'm +1.
Do you mind adding a empty-commit to trigger the CI-checks again ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 . The changes look good to me. Please resolve the failing CI checks.
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
Java doc issue is unrelated. Thanks @cxorm for the review. I'll merge the PR shortly. |
@xiaoyuyao @cxorm |
yeah, we do need a fix for this.... |
Thanks @steveloughran and @aajisaka. I will open a new ticket for the javadoc fix. |
#2802 is opened to fix the javadoc issue. |
apache#2762) (cherry picked from commit b503de2) Change-Id: I9957dd1b80720966829dd49cff133223b90829cf
…okenCach… (apache#2762)" This reverts commit 3102f84. Reason for revert: <INSERT REASONING HERE> Change-Id: I159965c018f89ba7e495f0467034ce1fc2ea8bdd
apache#2762) (cherry picked from commit b503de2) Change-Id: I25ced4aebca6092d4f08c1d9d5e355419bd36b38
https://issues.apache.org/jira/browse/HADOOP-17578
Additional Debug Log for troubleshooting TokenCache related issues.