dowenliu-xyz
dowenliu-xyz
从概率论上讲，代码里出现重名导致这个bug是一定会出现的。所以要讨论的是怎么处理这个bug。 要么就这么放着不处理，相当于认为是快速失败，那么遇到这个问题的团队能不能避免生产事故就要看研发有没有了解到这个bug、测试有没有测到这个场景了；否则就进行修复。 修复方案我的想法是既要避免报错，也保证流控不失效（不打破研发设计预期）。 关于复杂度，我没觉得这是个问题，而且我变更的代码我自己觉得还挺可控的，不复杂吧。😂 @LearningGp 你有其他的修复思路吗？ P.S. 欢迎试用插件。如果觉得不错，可以留言评论+分享。插件已设置开源项目 free 🤝。
> 如果方便的话，麻烦在sentinel-demo-annotation-spring-aop模块中完善下Demo以体现新的特性 OK，我这两天看时间搞下。
Done. Run module `sentinel-demo-annotation-spring-aop`, and run following requests (in curl format) to verify that fallbacks won't overlap each other. Once the module started, you can call requests in any sequence....
另外昨天还看了下 resilience4j 的做法。 https://github.com/resilience4j/resilience4j/blob/eeaf57a8217ded7fe14bad511454f263f9e6f06d/resilience4j-spring6/src/main/java/io/github/resilience4j/spring6/fallback/FallbackMethod.java#L85 他们的 `MethodMeta` 结构我觉得值得参考。（可以直接照抄😅）
Hi, I may be experiencing that problem as well. I started some mysql containers using docker compose. When I started the containers one by one and got to the 8th...
> Hi, I may be experiencing that problem as well. I started some mysql containers using docker compose. When I started the containers one by one and got to the...
这个问题的修复似乎与 #3386 类似，也要修改 `ResourceMetadataRegistry` 这个类。我现在不能确定是否代码冲突。等到 #3395 被合并或其他什么方式解决了 #3386 再处理这个 Bug。 我会在这几天先尝试完成 [idea 插件](https://plugins.jetbrains.com/plugin/24318-alibaba-sentinel-annotation-support/)对类级别注解的支持。
仔细分析了这个bug，不只是 class 级 annotation 会导致这个情况。 只要有多个不同返回值类型的方法的 `blockHandler` 、 `fallback` 、 `defaultFallback` 尝试使用同一处理类的相同名称的处理方法，都有可能会触发这个 bug 里。 具体来说： - `defaultFallback` 不同的方法尝试使用相同处理类中同名的处理方法，只要有一个方法的处理器方法能成功解析就必定触发这个 bug - `fallback` 、 `blockHandler` 不同的方法如果方法返回值不同（至少没有继承关系），如果尝试使用同一类中同名处理方法，只要有一个方法的处理器能成功解析就会触发这个 bug。 > 如果有多个解析成功，且它们的参数类型列表不同，会触发 #3386 - 不论是...
插件 [Alibaba Sentinel Annotation Support](https://plugins.jetbrains.com/plugin/24318-alibaba-sentinel-annotation-support) 版本已更新，对类注解提供了相关支持，同时对该bug情形进行了检查报错： https://github.com/alibaba/Sentinel/assets/7844338/859d41a2-e825-4b8f-8d18-bf91064c7cca
> 这个问题的修复似乎与 #3386 类似，也要修改 `ResourceMetadataRegistry` 这个类。我现在不能确定是否代码冲突。等到 #3395 被合并或其他什么方式解决了 #3386 再处理这个 Bug。 > > 我会在这几天先尝试完成 [idea 插件](https://plugins.jetbrains.com/plugin/24318-alibaba-sentinel-annotation-support/)对类级别注解的支持。 有谁能给我讲下，在类上使用注解的设计意图、实际场景吗？我在想是修复这个bug，还是提议干掉类注解。