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
HBASE-25642 Fix or stop warning about already cached block #3638
Conversation
Our logs have as a fairly common occurrence: 2021-03-05 22:24:31,034 WARN [StoreFileOpener-foo-1] hfile.BlockCacheUtil: Caching an already cached block: blah.bub. This is harmless and can happen in rare cases (see HBASE-8547) Because it is harmless, log at DEBUG level, not WARN.
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.
Do we need LOG. isDebugEnabled
condition ?
@shahrs87 No. In fact I was asked to remove such a guard on another PR. HBase 2+ uses logging libraries that do substitutions on '{}' only if the log level is enabled. So guards are no longer necessary. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
Our logs have as a fairly common occurrence: 2021-03-05 22:24:31,034 WARN [StoreFileOpener-foo-1] hfile.BlockCacheUtil: Caching an already cached block: blah.bub. This is harmless and can happen in rare cases (see HBASE-8547) Because it is harmless, log at DEBUG level, not WARN. Signed-off-by: Baiqiang Zhao <zhaobaiqiang@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Viraj Jasani <vjasani@apache.org> Reviewed-by: Rushabh Shah <rushabh.shah@salesforce.com>
Our logs have as a fairly common occurrence: 2021-03-05 22:24:31,034 WARN [StoreFileOpener-foo-1] hfile.BlockCacheUtil: Caching an already cached block: blah.bub. This is harmless and can happen in rare cases (see HBASE-8547) Because it is harmless, log at DEBUG level, not WARN. Signed-off-by: Baiqiang Zhao <zhaobaiqiang@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Viraj Jasani <vjasani@apache.org> Reviewed-by: Rushabh Shah <rushabh.shah@salesforce.com>
Our logs have as a fairly common occurrence: 2021-03-05 22:24:31,034 WARN [StoreFileOpener-foo-1] hfile.BlockCacheUtil: Caching an already cached block: blah.bub. This is harmless and can happen in rare cases (see HBASE-8547) Because it is harmless, log at DEBUG level, not WARN. Signed-off-by: Baiqiang Zhao <zhaobaiqiang@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Viraj Jasani <vjasani@apache.org> Reviewed-by: Rushabh Shah <rushabh.shah@salesforce.com>
Our logs have as a fairly common occurrence: 2021-03-05 22:24:31,034 WARN
[StoreFileOpener-foo-1] hfile.BlockCacheUtil: Caching an already cached
block: blah.bub. This is harmless and can happen in rare cases (see HBASE-8547)
Because it is harmless, log at DEBUG level, not WARN.