-
Notifications
You must be signed in to change notification settings - Fork 230
任务发放:适配 seata 框架:seata-solon-plugin #94
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
Comments
i will do it. |
我看长时间没有动作了,要不我来试一下? |
@xiaomizhou2 可以的,基于这个搞:https://github.com/noear/seata 已经搞了一半,只是没有熟悉的人使用和测试案例。。。。。。加 qq 或 微信,找我一下 |
好的 我看看 |
@fuzi1996 这个活。你要不要试试? |
好,我有空看看 |
@fuzi1996 代码可能已经差不多了。。。但我对它不熟,没法着力 |
我今天写了一个xa模式的demo,seata plugin没发现问题,但是使用的nami作为远调库有点问题,A调用B,如果B发生异常,nami获取到的返回值是null,但是在feign中如果B发生异常,请求状态码是500,此时feign的远调接口会报错,但是nami不会报错,这样会导致A事务本该回滚但没有回滚 |
@fuzi1996 照理。。。如果异常,会透传到 nami 的客户端的。。。代码,发我看下 |
@fuzi1996 收到,我研究下 |
你动作构快的。。。我这两天在搞 solon-ai-mcp |
这个修改很 ok 的(SolonApp 这个状态处理,有点历史原因。当有 StatusException 后,就可以直接 代码已合并。。。已发布:3.1.3-SNAPSHOT |
效果类型 “seata-spring-boot-starter”
The text was updated successfully, but these errors were encountered: