You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "keith-turner (via GitHub)" <gi...@apache.org> on 2024/09/23 17:34:47 UTC
[I] Remove LRU block cache [accumulo]
keith-turner opened a new issue, #4921:
URL: https://github.com/apache/accumulo/issues/4921
**Is your feature request related to a problem? Please describe.**
In #4920 the LRU block cache was deprecated and removed as the default and TinyLFU was set as the new deault.
**Describe the solution you'd like**
Remove the LRU block cache code if the TinyLFU default works well in all 4.0.0 testing.
The LRU code could be left as is. However its i a large amount of code and since its no longer the default it will not be exercised as much and therefore the confidence in its correctness will decrease over time as it is used less. So it would be good to remove it once the new default has been well exercised because its doubtful anyone will feel comfortable recommending its use.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Remove LRU block cache [accumulo]
Posted by "Amemeda (via GitHub)" <gi...@apache.org>.
Amemeda commented on issue #4921:
URL: https://github.com/apache/accumulo/issues/4921#issuecomment-5590076326
Looking into this, what is the status of 4.0.0 testing? Is it safe to remove LRU block cache now?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Remove LRU block cache [accumulo]
Posted by "ddanielr (via GitHub)" <gi...@apache.org>.
ddanielr commented on issue #4921:
URL: https://github.com/apache/accumulo/issues/4921#issuecomment-5597061622
I'm not aware of any issues with the block cache when testing with 4.0.0.
Users are able to change the block cache class via a property since version 2.0.0. so they do have a chance to test TinyLFU today without needing to go to 4.0.0
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Remove LRU block cache [accumulo]
Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii commented on issue #4921:
URL: https://github.com/apache/accumulo/issues/4921#issuecomment-5736572643
I'm not too worried about removing it now. I haven't seen any issues with the default. But, if we do find issues, it's trivial to bring it back or provide an alternate implementation in a supplemental jar to add to the classpath.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org