bruce

org.apache.dubbo.config.ApplicationConfig#0

1

Invalid name="org.apache.dubbo.config.ApplicationConfig#0" contains illegal character, only digit, letter, '-', '_' or '.' is legal.

feature: 配置集群限流时候,建议有个单机的阈值兜底. 目前的如果选择集群限流后, count是集群限流的阈值, 如果出现降级, 使用的是集群限流的阈值, 建议加一个单机阈值, 降级时候使用单机阈值,而不是集群阈值

希望能够提供集群限流 Token Server压力测试的数据

使用dubbo原生的属性值获取consumer appName

2

sentinel适配dubbo, 获取消费端appname的方式可能是上上游的应用名, 应该直接用dubbo的remote.application属性获取

Can it support dynamically determining whether to enable gzip compression based on message size

1

Can it support dynamically determining whether to enable gzip compression based on message size ?

[BUG] 指标文件读取方法不正确

3

https://github.com/alibaba/Sentinel/blob/195150bc745927429e9f14f501907310b46d702f/sentinel-core/src/main/java/com/alibaba/csp/sentinel/node/metric/MetricSearcher.java#L208-L211 209行 offset = indexIn.readLong(); 不应该写, 应该剔除这行, 这样会导致读取的数据不正确 假如读取的是 1729965161000L的数据, 返回的是1729965162000L及之后的数据, 但如果1729965161000L是文件的最后一个时间的记录数据, 又会被读取到 ``` List metricNodes = searcher.find(1729965161000L, 5); ```

MetricNode#toFatString 时间转换影响性能

4

https://github.com/alibaba/Sentinel/blob/195150bc745927429e9f14f501907310b46d702f/sentinel-core/src/main/java/com/alibaba/csp/sentinel/node/metric/MetricNode.java#L213-L230 214行, 每次都创建新的SimpleDateFormat影响性能, 改成java8的时间吧

Dubbo adapter 使用固定的 sentinel_dubbo_context 入口是不是好一些？

1

这里的context name 固定为 `sentinel_dubbo_context` 是不是更好点, 类似`com.alibaba.csp.sentinel.adapter.servlet.CommonFilter`中的`sentinel_web_servlet_context` 也是这种用法  源码位置 https://github.com/alibaba/Sentinel/blob/d00798ff8d04c805a0f64772d7802b801e1c86e3/sentinel-adapter/sentinel-apache-dubbo-adapter/src/main/java/com/alibaba/csp/sentinel/adapter/dubbo/SentinelDubboProviderFilter.java#L76

https://github.com/alibaba/Sentinel/blob/195150bc745927429e9f14f501907310b46d702f/sentinel-core/src/main/java/com/alibaba/csp/sentinel/node/metric/MetricsReader.java#L54-L56 55行没必要为了获取时间戳而将字符串转成``MetricNode``对象, 直接通过字符串截取就行, ```java int timestampIndex = s.indexOf("|"); String timestamp = Long.parseLong(s.substring(0, timestampIndex)); ``` https://github.com/alibaba/Sentinel/blob/195150bc745927429e9f14f501907310b46d702f/sentinel-transport/sentinel-transport-common/src/main/java/com/alibaba/csp/sentinel/command/handler/SendMetricCommandHandler.java#L97-L100 SendMetricCommandHandler中为了得到`ThinString`, 又将`MetricNode`转成字符串, `MetricNode`成了冗余转换, 会增加内存占用 不如直接采用`FatString`字符串分割 ```java int timestampIndex = s.indexOf("|"); String timestamp = s.substring(0, timestampIndex);...

fix issue : https://github.com/spring-cloud/spring-cloud-openfeign/issues/1211