-
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-26659 The ByteBuffer of metadata in RAMQueueEntry in BucketCache could be reused. #4026
Conversation
…e could be reused.
🎊 +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. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
// safely. Otherwise, this reuse is problematic. Fortunately, the data is already | ||
// transferred with our current IOEngines. Should take care, when we have new types of | ||
// IOEngine in the future. | ||
metaBuff.clear(); |
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.
Since the metaBuff is passed in from upper layer, I prefer we also do this in upper layer? And what if the above ioEngine.write throws IOException out?
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.
Have made the metaBuff cleared before invoking of #writeToCache. I think this can keep the metaBuff clean every time we writeToCache even if the IOEngine had exception and failed the last block write.
🎊 +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. |
🎊 +1 overall
This message was automatically generated. |
…e could be reused. (#4026) Signed-off-by: Duo Zhang <zhangduo@apache.org>
…e could be reused. (#4026) Signed-off-by: Duo Zhang <zhangduo@apache.org>
https://issues.apache.org/jira/browse/HBASE-26659