-
Notifications
You must be signed in to change notification settings - Fork 9.2k
HDFS-16425. Add debug log when requeue observer read call #3886
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
HDFS-16425. Add debug log when requeue observer read call #3886
Conversation
|
💔 -1 overall
This message was automatically generated. |
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/Server.java
Outdated
Show resolved
Hide resolved
858131d to
cde094c
Compare
|
💔 -1 overall
This message was automatically generated. |
cde094c to
8476b8c
Compare
|
💔 -1 overall
This message was automatically generated. |
ayushtkn
left a comment
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.
Jenkins seems not happy:
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3886/3/artifact/out/blanks-eol.txt
8476b8c to
112f12e
Compare
@ayushtkn , Hi , i feel so sorry for replying so late. I have fixed the style problems. |
|
💔 -1 overall
This message was automatically generated. |
112f12e to
d9ed717
Compare
|
💔 -1 overall
This message was automatically generated. |
|
@ayushtkn Hi~ Ayush. Could you please help to review the code, thanks a lot. |
ayushtkn
left a comment
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.
I think logging call backfired here in HADOOP-18082. Can you folks give a check. There is a mail thread as well, telling not to log call.
Hi, Ayush. Very thanks for your opinions. I will learn the context in HADOOP-18082 and HADOOP-18143 |
|
We're closing this stale PR because it has been open for 100 days with no activity. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable. |
Description of PR
Add debug log when requeue observer read call
How was this patch tested?
no need to test.