-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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-28189 Fix the miss count in one of CombinedBlockCache getBlock implementations #5506
Conversation
…implementations Change-Id: I9edff6fc2ad88128a4f695bd7477d2fca5bc28a8
💔 -1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
💔 -1 overall
This message was automatically generated. |
…l type of block Change-Id: Ia3354a4801ae505963f7515b1324ecddf4dd97dc
🎊 +1 overall
This message was automatically generated. |
boolean metaBlock = isMetaBlock(cacheKey.getBlockType()); | ||
if (metaBlock) { | ||
if (!existInL1 && block != null) { | ||
LOG.warn("Cache key {} had block type {}, but was found in L2 cache.", cacheKey, |
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.
How can this happen? Is this a bug?
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 this is just a defensive logging? It should not happen normally. If we see this log, then there could be a bug?
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.
Yes, I decided to put this as I saw a UT doing the opposite (creating a meta block then wrongly passing block type data in the cachekey), so I wonder if this mistake might be happening anywhere in the "real" code. I tried to some review of all callees to this method and haven't found it, but I could had missed something.
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
…implementations (#5506) Signed-off-by: Peter Somogyi <psomogyi@apache.org>
…implementations (#5506) Signed-off-by: Peter Somogyi <psomogyi@apache.org> Change-Id: I472b6019c3fe744f7afc4219e47967a0d5823c6f
…implementations (#5506) Signed-off-by: Peter Somogyi <psomogyi@apache.org>
…implementations (#5506) Signed-off-by: Peter Somogyi <psomogyi@apache.org>
…he getBlock implementations (apache#5506) Signed-off-by: Peter Somogyi <psomogyi@apache.org> Change-Id: Icee78a9c839b3c4e4aac2176aa4091cdcd47a08b
No description provided.