⚠️ 验证
🎯 方案描述
hystrix filter 已在 dubbo-go 主仓的 #3253 中迁移到 apache/dubbo-go-extensions,当前实现位置为 filter/hystrix/filter.go。
为了减少维护者和使用方的理解偏差,建议明确 hystrix filter 在并发调用场景下的并发约束:
- 明确说明该 filter 是否支持多个 goroutine 并发调用
Invoke。
- 若存在并发前提或限制,请在 README / 代码注释中显式写清。
- 如有必要,再评估是否在入口增加最小化串行保护(需 maintainer 先确认是否接受该方向)。
📋 使用场景
- 在高并发场景启用
hystrix_consumer / hystrix_provider 的用户,需要明确的线程安全预期。
- 评审 race 问题时,需要有明确并发契约来判断是实现缺陷还是调用方越界使用。
⚖️ 复杂度与风险
- 先做文档/注释澄清,风险低,且能提升评审一致性。
- 如果加运行时串行保护,可能影响吞吐,需要单独评估并达成共识后再实现。
🔗 外部依赖
github.com/afex/hystrix-go/hystrix 的行为与并发保证。
📘 补充上下文
如果 maintainer 认可这个方向,我可以继续补一个小 PR,优先先做文档/注释层的并发约束说明。
🎯 方案描述
hystrix filter 已在 dubbo-go 主仓的 #3253 中迁移到
apache/dubbo-go-extensions,当前实现位置为filter/hystrix/filter.go。为了减少维护者和使用方的理解偏差,建议明确 hystrix filter 在并发调用场景下的并发约束:
Invoke。📋 使用场景
hystrix_consumer/hystrix_provider的用户,需要明确的线程安全预期。⚖️ 复杂度与风险
🔗 外部依赖
github.com/afex/hystrix-go/hystrix的行为与并发保证。📘 补充上下文
如果 maintainer 认可这个方向,我可以继续补一个小 PR,优先先做文档/注释层的并发约束说明。