alibaba/Sentinel

Replace deprecated concurrentlinkedhashmap-lru with Caffeine in param-flow-extension

Open

#2,871 opened on Sep 8, 2022

View on GitHub
 (4 comments) (0 reactions) (0 assignees)Java (23,109 stars) (8,150 forks)batch import
good first issuekind/enhancement

Description

Issue Description

Type: bug report or feature request

Describe what happened (or what feature you want)

concurrentlinkedhashmap-lru 已经很老了，没有维护。

Describe what you expected to happen

是不是可以考虑下使用Caffeine 替换。

Caffeine is the Java 8 successor to ConcurrentLinkedHashMap and Guava's cache. Projects should prefer Caffeine and migrate when requiring JDK8 or higher. The previous caching projects are supported in maintenance mode.

How to reproduce it (as minimally and precisely as possible)

Tell us your environment

Anything else we need to know?

Contributor guide

Tech stack
java
Domain
backend
Issue type
refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
mostly clear
Prerequisites
Java 8Maven buildBasic knowledge of caching
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70
Research direction
Locate the param flow extension module in the Sentinel repository. Replace all usages of `ConcurrentLinkedHashMap` and related classes from the concurrentlinkedhashmap lru library with Caffeine's `Caffeine` and `Cache` classes. Ensure that the caching behavior (e.g., eviction policies) remains equivalent. Review existing tests to verify functionality. No linked PRs or specific files are mentioned; you may need to explore the module's structure.