-
Notifications
You must be signed in to change notification settings - Fork 8.8k
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
1.8.0版本XA模式如何支持达梦数据库 #6367
Comments
check #3672 now it is at mode |
XA模式如何支持 |
Can I submit a supported pr and you can help me to test it? Because I don't have a dameng database environment |
public static XAConnection createXAConnection(Connection physicalConn, Driver driver, String dbType) throws SQLException { |
如果你读懂了这些代码的意思,并且知道dameng如何创建xaconnection,你就可以提交一个pr来进行支持,比如你可以参考mariadbdb是如何被支持的 |
代码没有问题,我是希望SEATA可以留有扩展点,用于自行实现,而不是这样固定代码 |
You are welcome to submit a pr to transform createxaconnection into an extended implementation by way of spi. |
The spi may have to consider how different database implementations can be common, for example, sqlserver does not support building xaconnection through the connection of a normal datasource, it can only create xaconnection by building xadatasource. |
扩展可以保留吧,是否可以再更高层进行扩展,而不是在util中进行处理 |
You can look at seata's xa implementation, in fact, there are not many extensible points, because the final implementation of xa for each database has some deviation, such as oracle and mysql are different, mysql high version relative to the current mariadb and different, in the first stage and second stage processing is also different, which in the adaption of a more messy, not very easy to carry out a unified abstraction |
Ⅰ. Issue Description
1.8.0版本XA模式如何支持达梦数据库,以及其它支持XA的数据库
Ⅱ. Describe what happened
If there is an exception, please attach the exception trace:
Ⅲ. Describe what you expected to happen
Ⅳ. How to reproduce it (as minimally and precisely as possible)
Minimal yet complete reproducer code (or URL to code):
Ⅴ. Anything else we need to know?
Ⅵ. Environment:
java -version
):uname -a
):The text was updated successfully, but these errors were encountered: