Skip to content

任务发放:适配 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

Open
noear opened this issue Apr 5, 2023 · 14 comments
Open

任务发放:适配 seata 框架:seata-solon-plugin #94

noear opened this issue Apr 5, 2023 · 14 comments
Assignees
Labels
enhancement New feature or request

Comments

@noear
Copy link
Member

noear commented Apr 5, 2023

效果类型 “seata-spring-boot-starter”

@noear noear added enhancement New feature or request good first issue Good for newcomers labels Apr 5, 2023
@noear noear removed the good first issue Good for newcomers label May 8, 2023
@tobehardest
Copy link

i will do it.

@xiaomizhou2
Copy link

我看长时间没有动作了,要不我来试一下?

@noear
Copy link
Member Author

noear commented Jan 23, 2025

@xiaomizhou2 可以的,基于这个搞:https://github.com/noear/seata

已经搞了一半,只是没有熟悉的人使用和测试案例。。。。。。加 qq 或 微信,找我一下

@xiaomizhou2
Copy link

好的 我看看

@noear
Copy link
Member Author

noear commented Mar 11, 2025

@fuzi1996 这个活。你要不要试试?

@fuzi1996
Copy link
Contributor

好,我有空看看

@noear
Copy link
Member Author

noear commented Mar 12, 2025

@fuzi1996 代码可能已经差不多了。。。但我对它不熟,没法着力

@fuzi1996
Copy link
Contributor

fuzi1996 commented Apr 6, 2025

我今天写了一个xa模式的demo,seata plugin没发现问题,但是使用的nami作为远调库有点问题,A调用B,如果B发生异常,nami获取到的返回值是null,但是在feign中如果B发生异常,请求状态码是500,此时feign的远调接口会报错,但是nami不会报错,这样会导致A事务本该回滚但没有回滚

@noear
Copy link
Member Author

noear commented Apr 6, 2025

@fuzi1996 照理。。。如果异常,会透传到 nami 的客户端的。。。代码,发我看下

@noear
Copy link
Member Author

noear commented Apr 6, 2025

@fuzi1996 收到,我研究下

@fuzi1996
Copy link
Contributor

fuzi1996 commented Apr 8, 2025

我追踪到这个jar包

Image

代码位置: org.noear.solon.core.mvc.ActionDefault#invoke

Image

render时 抛出异常

org.noear.solon.serialization.StringSerializerRender#render

Image

最终 org.noear.solon.SolonApp#tryHandle

Image

不满足条件,没有任何处理

我想改成这个样子,报错但是又没处理直接 500

Image

如果可以,我就提pr

@noear
Copy link
Member Author

noear commented Apr 8, 2025

你动作构快的。。。我这两天在搞 solon-ai-mcp

@noear
Copy link
Member Author

noear commented Apr 8, 2025

这个修改很 ok 的(SolonApp 这个状态处理,有点历史原因。当有 StatusException 后,就可以直接 x.status(500); 了)。

代码已合并。。。已发布:3.1.3-SNAPSHOT

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants