phuslu / lru

High performance LRU cache
MIT License
205 stars 8 forks source link

fix invalid cache length when set with zero value key #24

Open godruoyi opened 1 week ago

godruoyi commented 1 week ago

Description

fix #17

The main change is to delete the previously existing key when the list is full. Note that these changes work fine with single shard only, there still have some issues when we using multiple shards. will fix them in the next PR.

phuslu commented 1 week ago

thanks! let me look into it.

godruoyi commented 19 hours ago

Hi @phuslu, are there any updates on this? Please feel free to ask me for more context or more updates if needed.