You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by GitBox <gi...@apache.org> on 2020/03/29 03:57:06 UTC
[GitHub] [incubator-shardingsphere] cherrylzhao opened a new issue #4991:
make ShardingDataSource compatible with Seata in micro service distribution
transaction
cherrylzhao opened a new issue #4991: make ShardingDataSource compatible with Seata in micro service distribution transaction
URL: https://github.com/apache/incubator-shardingsphere/issues/4991
In micro-service distribution transaction, Seata will wrap `ShardingDataSource` as `DataSourceProxy` which communication with seata-server through RPC protocol,
and DataSourceProxy will invoke some standard JDBC interface to get metadata from ShardingDataSource, however ShardingDataSource don't compatible with it. I have investigated in my local environment and found these issues we should resolve
- [ ] support get information by logical schema in CachedDataBaseMetaData
- [ ] support getParameterMetaData in ShardingPreparedStatement
- [ ] add `client.transaction.micro-service-enable` in sharding-transaction-base-seata-at
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-shardingsphere] cherrylzhao commented on issue #4991:
make ShardingDataSource compatible with Seata in micro-service distribution
transaction
Posted by GitBox <gi...@apache.org>.
cherrylzhao commented on issue #4991: make ShardingDataSource compatible with Seata in micro-service distribution transaction
URL: https://github.com/apache/incubator-shardingsphere/issues/4991#issuecomment-605644192
I have submitted a PR for this config see #4996
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-shardingsphere] cherrylzhao closed issue #4991: make
ShardingDataSource compatible with Seata in micro-service distribution
transaction
Posted by GitBox <gi...@apache.org>.
cherrylzhao closed issue #4991: make ShardingDataSource compatible with Seata in micro-service distribution transaction
URL: https://github.com/apache/incubator-shardingsphere/issues/4991
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-shardingsphere] cherrylzhao commented on issue #4991:
make ShardingDataSource compatible with Seata in micro-service distribution
transaction
Posted by GitBox <gi...@apache.org>.
cherrylzhao commented on issue #4991: make ShardingDataSource compatible with Seata in micro-service distribution transaction
URL: https://github.com/apache/incubator-shardingsphere/issues/4991#issuecomment-605626684
here is the config for integrating seata-at currently in shardingspehre side
```
client {
application.id = test
transaction.service.group = my_test_tx_group
}
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-shardingsphere] cherrylzhao commented on issue #4991:
make ShardingDataSource compatible with Seata in micro-service distribution
transaction
Posted by GitBox <gi...@apache.org>.
cherrylzhao commented on issue #4991: make ShardingDataSource compatible with Seata in micro-service distribution transaction
URL: https://github.com/apache/incubator-shardingsphere/issues/4991#issuecomment-605627436
how about this ?
```
sharding.transaction.seata.at.enable = true
client {
application.id = test
transaction.service.group = my_test_tx_group
}
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-shardingsphere] kimmking commented on issue #4991: make
ShardingDataSource compatible with Seata in micro-service distribution
transaction
Posted by GitBox <gi...@apache.org>.
kimmking commented on issue #4991: make ShardingDataSource compatible with Seata in micro-service distribution transaction
URL: https://github.com/apache/incubator-shardingsphere/issues/4991#issuecomment-605603672
`enable.client.transaction.in.seata.at` is better?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [shardingsphere] dengjin001 commented on issue #4991: make ShardingDataSource compatible with Seata in micro-service distribution transaction
Posted by GitBox <gi...@apache.org>.
dengjin001 commented on issue #4991:
URL: https://github.com/apache/shardingsphere/issues/4991#issuecomment-616339246
seata-1.1.0 is supports *.yml , so `sharding.transaction.seata.at.enable = true` support yml configuration?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-shardingsphere] cherrylzhao commented on issue #4991:
make ShardingDataSource compatible with Seata in micro-service distribution
transaction
Posted by GitBox <gi...@apache.org>.
cherrylzhao commented on issue #4991: make ShardingDataSource compatible with Seata in micro-service distribution transaction
URL: https://github.com/apache/incubator-shardingsphere/issues/4991#issuecomment-605624611
kimmking, thanks for suggestion.
sharding-transaction-base-seata-at module only worked in none-microservice mode since it invoke the seata global transaction api directly, so I want to add a switcher to disable it if user want to introduce seata global transaction in microservice. also I'm consider about better way to implement that
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-shardingsphere] kimmking commented on issue #4991: make
ShardingDataSource compatible with Seata in micro-service distribution
transaction
Posted by GitBox <gi...@apache.org>.
kimmking commented on issue #4991: make ShardingDataSource compatible with Seata in micro-service distribution transaction
URL: https://github.com/apache/incubator-shardingsphere/issues/4991#issuecomment-605759167
it's awesome
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [shardingsphere] cay914 edited a comment on issue #4991: make ShardingDataSource compatible with Seata in micro-service distribution transaction
Posted by GitBox <gi...@apache.org>.
cay914 edited a comment on issue #4991:
URL: https://github.com/apache/shardingsphere/issues/4991#issuecomment-669044265
@cherrylzhao where to config **sharding.transaction.seata.at.enable = true** ?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [shardingsphere] cay914 commented on issue #4991: make ShardingDataSource compatible with Seata in micro-service distribution transaction
Posted by GitBox <gi...@apache.org>.
cay914 commented on issue #4991:
URL: https://github.com/apache/shardingsphere/issues/4991#issuecomment-669044265
@cherrylzhao where to config sharding.transaction.seata.at.enable = true ?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org