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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to