apache / apache/accumulo

Remove LRU block cache

Open
#4,921 2 comments 0 reactions 0 assignees View on GitHub
blocker enhancement
Dominant language
Java
Stars
1.2k
Forks
487
Avg merge
4d 5h
Merged PRs (30d)
13

Description

**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 and the codebase changes around it. 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.

Contributor guide

Open the contributing guide

Research direction

Start by reviewing issue #4920 and the 4.0.0 testing that exercises the TinyLFU default. Locate the LRU block cache implementation and its related references, then confirm the full test suite passes without it. Done means the obsolete LRU code and references are removed while TinyLFU remains the working default.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
database, performance
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.