-
Notifications
You must be signed in to change notification settings - Fork 3k
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
txmgr: Simplify API #5306
txmgr: Simplify API #5306
Conversation
|
9d09d96
to
bf769d8
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #5306 +/- ##
===========================================
- Coverage 39.31% 35.29% -4.02%
===========================================
Files 389 225 -164
Lines 25056 20052 -5004
Branches 838 0 -838
===========================================
- Hits 9851 7078 -2773
+ Misses 14469 12295 -2174
+ Partials 736 679 -57
Flags with carried forward coverage won't be shown. Click here to find out more.
|
90c5955
to
18524db
Compare
Hey @trianglesphere! This PR has merge conflicts. Please fix them before continuing review. |
bf769d8
to
d8a1f2b
Compare
✅ Deploy Preview for opstack-docs canceled.
|
d8a1f2b
to
0b72447
Compare
This PR has been added to the merge queue, and will be merged soon. |
This PR is next in line to be merged, and will be merged as soon as checks pass. |
Description
This simplifies some of the transaction manager APIs.
TODOs