-
Notifications
You must be signed in to change notification settings - Fork 3.4k
HBASE-26154: Provide exception metric for quota exceeded and throttling #3544
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
e0515c3
to
454f2e0
Compare
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
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.
A minor comment, else LGTM.
hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/MetricsHBaseServer.java
Outdated
Show resolved
Hide resolved
454f2e0
to
40e70ba
Compare
Thank you both for the review. I resolved the one comment, so this is ready for merge once the build succeeds. I think this should apply cleanly to branch-2.4 if you'd be willing as part of the merge. |
hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/MetricsHBaseServer.java
Outdated
Show resolved
Hide resolved
40e70ba
to
32bc937
Compare
Do you mind raising PR for master branch? We apply changes first to master branch if applicable and then backport, the only exception to this practice are for changes that are not applicable to the newer release lines |
🎊 +1 overall
This message was automatically generated. |
@pankaj72981 I've submitted #3545 for master and #3546 for branch 2.4 |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
…cThrottlingException
32bc937
to
766f9c3
Compare
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
See https://issues.apache.org/jira/browse/HBASE-26154
Also adds a debug log for unknown exceptions, which we've also found helpful.