-
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-27750: Update the list of prefetched Hfiles upon block eviction #5140
Conversation
@@ -119,21 +120,36 @@ public void testPrefetchPersistenceCrash() throws Exception { | |||
|
|||
@Test | |||
public void testPrefetchPersistenceCrashNegative() throws Exception { | |||
long bucketCachePersistInterval = 3000; | |||
long bucketCachePersistInterval = 5000; |
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.
Actually, we should make BucketCachePersister a daemon thread. Then we could set this interval to Long.MAX_VALUE.
🎊 +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. |
Latest UT failure seems unrelated to me, but have re-triggered the pre-commit job. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
…#5140) Signed-off-by: Wellington Chevreuil <wchevreuil@apache.org>
…les upon block eviction (apache#5140) Signed-off-by: Wellington Chevreuil <wchevreuil@apache.org> Change-Id: Ifdfa755a800c1c926403b90ec991d7d2e9f7da38
Currently, we maintain a list of Hfiles on disk for which prefetch is complete and avoid prefetching those files after a restart. But we don't handle cases where blocks are evicted from the cache. This ticket is for updating the list of prefetched files upon simple block eviction.
Jira: https://issues.apache.org/jira/browse/HBASE-27750