4.5K Star 42.8K Fork 12.6K

GVPlengleng / pig

 / 详情

sentinel 异常降级失效

已完成
创建于  
2020-06-29 16:42

pigx 相关问题请去此提,码云忽略

版本信息

  • pig版本: 2.8

  • 是否修改包名: 无

报错信息

测试使用sentinel 界面管理上面的 异常数量降级失效

评论 (5)

妖孽 创建了任务
妖孽 关联仓库设置为pig4cloud.com/pig
展开全部操作日志

看了一下不知道这样解决可以不,有生效 输入图片说明
输入图片说明

输入图片说明

https://gitee.com/log4j/pig/blob/master/pig-common/pig-common-sentinel/src/main/java/com/pig4cloud/pig/common/sentinel/feign/PigSentinelInvocationHandler.java#L104

spring cloud sentinel 处理的场景是 feign 远程调用的降级异常 tracer 记录了。

这种普通的异常 ,一般情况下不会产生服务雪崩的 问题 。

这里加上 那个模块要依赖 sentinel 就耦合了 看看咋个处理更好一点

https://gitee.com/log4j/pig/blob/master/pig-common/pig-common-sentinel/src/main/java/com/pig4cloud/pig/common/sentinel/feign/PigSentinelInvocationHandler.java#L104
spring cloud sentinel 处理的场景是 feign 远程调用的降级异常 tracer 记录了。
这种普通的异常 ,一般情况下不会产生服务雪崩的 问题 。
这里加上 那个模块要依赖 sentinel 就耦合了 看看咋个处理更好一点

@lengleng 有道理。。。 不过普通服务完全有降级的需求

:+1: 嗯 那就把全局的 异常处理 放到 common-sentinel 里面处理,避免耦合

这样改动会有个问题 ,auth 模块的异常被 global 处理,而不是oauht2 的处理机制。 我看看如何改动 最好

@妖孽 感谢分享

lengleng 添加了
 
新功能
标签
lengleng 任务状态待办的 修改为已完成

登录 后才可以发表评论

状态
负责人
里程碑
Pull Requests
关联的 Pull Requests 被合并后可能会关闭此 issue
分支
开始日期   -   截止日期
-
置顶选项
优先级
参与者(2)
410595 log4j 1591768487
Java
1
https://gitee.com/log4j/pig.git
git@gitee.com:log4j/pig.git
log4j
pig
pig

搜索帮助