-
Notifications
You must be signed in to change notification settings - Fork 8.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support LRU evicting mechanism for origin statistic map in StatisticNode #860
Comments
According to the caller to limit the flow, I hope to support the LRU, which can flexibly support the current limit by various sources, without worrying about memory problems.Issue DescriptionType: feature request Describe what happened (or what feature you want)In our business scenario, we hope to be able to limit traffic according to the IP of the caller. Sentinel currently supports traffic limiting according to the calling source, but there is no elimination strategy. The service is actually used. If the number of IPs is large, the risk is a bit large, so I hope to be able to Support LRU, the business is more stable to use. Describe what you expected to happenHow to reproduce it (as minimally and precisely as possible)Tell us your environmentAnything else we need to know? |
This could bring extra dependencies (e.g. ConcurrentLinkedHashMap), but it's might be necessary for some circumstances. What do you think of it? @CarpenterLee @jasonjoo2010 @cdfive |
这个功能什么时候能够合并到主版本?非常有用! |
* Remove the useless files * Replace PermSize with MetaspaceSize, details see http://openjdk.java.net/jeps/122 * Update DLedgerCommitLog.java (alibaba#1145) Delete useless code * Remove the duplicate content * Polish the comment (alibaba#1107) * Minor Typo fix (alibaba#860) * [ISSUE alibaba#1082] Fix disconnection of HA (alibaba#1083) * fixed the text description in chinese doc (alibaba#1339) * fix /dev/shm not found on some OSs (alibaba#1345) * Refactor the protection logic when pulling * change the MQVersion variable to rocketmq 4.5.2 version; * Minor polish * Fix the wrong package name * [maven-release-plugin] prepare release rocketmq-all-4.5.2 * [maven-release-plugin] prepare for next development iteration * [RIP-15]Add Ipv6 support for RocketMQ
Issue Description
Type: feature request
Describe what happened (or what feature you want)
我们的业务场景,希望能能够按照调用方的IP来限流,Sentinel 目前支持按照调用源来限流,但是没有淘汰策略,业务真正使用起来,假如IP数量比较多,风险有点大,所以希望能够支持LRU,业务使用起来更稳定。
其实热点参数限流,目前是支持LRU的,但是为什么不能用热点参数来限流呢?
热点参数限流,一般情况下,需要把限流的源绑定到参数上,这样才可以使用。但是实际的业务场景很复杂,例如上游服务不愿意改动,所以用热点参数限流可能做起来就不是那么的方便。所以才需要能够按照调用源来限流。例如按IP,这样只需要获取到上游服务的IP就行,并且IP过多的情况下,还支持LRU,就比较稳妥了。
Describe what you expected to happen
How to reproduce it (as minimally and precisely as possible)
Tell us your environment
Anything else we need to know?
The text was updated successfully, but these errors were encountered: