Skip to content
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

<perf>(xatransaction): Improve the xaTransaction list interface #596

Merged
merged 5 commits into from
Dec 15, 2023

Conversation

yinghuochongfly
Copy link
Contributor

@yinghuochongfly yinghuochongfly commented Dec 5, 2023

(xatransaction): Improve the xaTransaction list interface

@codecov-commenter
Copy link

codecov-commenter commented Dec 13, 2023

Codecov Report

Merging #596 (696e0a6) into feature-1.4.0 (8d3b6b3) will decrease coverage by 0.06%.
The diff coverage is 0.00%.

❗ Current head 696e0a6 differs from pull request most recent head e1e9770. Consider uploading reports for the commit e1e9770 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@                 Coverage Diff                 @@
##             feature-1.4.0     #596      +/-   ##
===================================================
- Coverage            15.10%   15.04%   -0.06%     
  Complexity             441      441              
===================================================
  Files                  186      186              
  Lines                 8972     9005      +33     
  Branches               764      768       +4     
===================================================
  Hits                  1355     1355              
- Misses                7499     7532      +33     
  Partials               118      118              
Files Coverage Δ
...ross/network/rpc/handler/XATransactionHandler.java 4.32% <0.00%> (-0.11%) ⬇️
...ebank/wecross/routine/xa/XATransactionManager.java 0.00% <0.00%> (ø)

@kyonRay kyonRay merged commit 6def3f7 into WeBankBlockchain:feature-1.4.0 Dec 15, 2023
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants