Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Sharding proxy xa transaction #974

Merged
merged 35 commits into from
Jun 29, 2018
Merged

Sharding proxy xa transaction #974

merged 35 commits into from
Jun 29, 2018

Conversation

cherrylzhao
Copy link
Member

for #902 add proxy-xa-transaction feature

resolve can't get transaction execute result within a channel.
…into sharding-proxy-xa-transaction

# Conflicts:
#	sharding-proxy/src/main/java/io/shardingsphere/proxy/backend/common/SQLExecuteBackendHandler.java
#	sharding-proxy/src/main/java/io/shardingsphere/proxy/backend/common/SQLExecuteWorker.java
#	sharding-proxy/src/main/java/io/shardingsphere/proxy/backend/common/StatementExecuteWorker.java
#	sharding-proxy/src/main/java/io/shardingsphere/proxy/transport/mysql/packet/command/text/query/ComQueryPacket.java
 into sharding-proxy-xa-transaction

# Conflicts:
#	sharding-proxy/src/main/java/io/shardingsphere/proxy/backend/common/SQLExecuteBackendHandler.java
#	sharding-proxy/src/main/java/io/shardingsphere/proxy/backend/common/SQLExecuteWorker.java
#	sharding-proxy/src/main/java/io/shardingsphere/proxy/backend/common/StatementExecuteBackendHandler.java
#	sharding-proxy/src/main/java/io/shardingsphere/proxy/backend/common/StatementExecuteWorker.java
 into sharding-proxy-xa-transaction

# Conflicts:
#	sharding-proxy/src/main/java/io/shardingsphere/proxy/backend/common/SQLExecuteBackendHandler.java
#	sharding-proxy/src/main/java/io/shardingsphere/proxy/backend/common/StatementExecuteBackendHandler.java
#	sharding-proxy/src/main/java/io/shardingsphere/proxy/backend/common/StatementExecuteWorker.java
@codecov-io
Copy link

Codecov Report

Merging #974 into dev will increase coverage by 0.62%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##              dev    #974      +/-   ##
=========================================
+ Coverage   54.88%   55.5%   +0.62%     
=========================================
  Files         579     585       +6     
  Lines       12173   12096      -77     
  Branches     1911    1924      +13     
=========================================
+ Hits         6681    6714      +33     
+ Misses       5098    4991     -107     
+ Partials      394     391       -3
Impacted Files Coverage Δ
...re/proxy/metadata/ProxyShardingRefreshHandler.java 0% <ø> (ø) ⬆️
...re/proxy/backend/common/ExecuteBackendHandler.java 0% <0%> (ø)
...rdingsphere/proxy/config/XaProxyRawDataSource.java 0% <0%> (ø) ⬆️
...e/proxy/backend/common/StatementExecuteWorker.java 0% <0%> (ø) ⬆️
...proxy/backend/common/SQLExecuteBackendHandler.java 0% <0%> (ø) ⬆️
...oxy/backend/resource/ProxyJDBCResourceFactory.java 0% <0%> (ø)
...gsphere/proxy/frontend/common/FrontendHandler.java 0% <0%> (ø) ⬆️
...ysql/packet/command/text/query/ComQueryPacket.java 0% <0%> (ø) ⬆️
...here/proxy/backend/resource/ProxyJDBCResource.java 0% <0%> (ø)
...phere/proxy/backend/resource/BaseJDBCResource.java 0% <0%> (ø)
... and 16 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d0eaa53...06231c5. Read the comment docs.

@terrymanu terrymanu merged commit bd7ce2f into dev Jun 29, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants