Xline icon indicating copy to clipboard operation
Xline copied to clipboard

feat: implement rocks transaction

Open bsbds opened this issue 3 months ago • 3 comments

Depends-On: #674

Please briefly answer these questions:

  • what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?)

    Implement txn API for rocksdb engine.

  • what changes does this pull request make?

  • are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc)

bsbds avatar Mar 18 '24 02:03 bsbds

Codecov Report

Attention: Patch coverage is 15.96244% with 179 lines in your changes are missing coverage. Please review.

Project coverage is 75.11%. Comparing base (e35b35a) to head (44a5319). Report is 49 commits behind head on master.

:exclamation: Current head 44a5319 differs from pull request most recent head 7e010f6. Consider uploading reports for the commit 7e010f6 to get more accurate results

Files Patch % Lines
crates/engine/src/rocksdb_engine/transaction.rs 15.56% 175 Missing and 4 partials :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #703      +/-   ##
==========================================
- Coverage   75.55%   75.11%   -0.44%     
==========================================
  Files         180      187       +7     
  Lines       26938    27772     +834     
  Branches    26938    27772     +834     
==========================================
+ Hits        20353    20861     +508     
- Misses       5366     5630     +264     
- Partials     1219     1281      +62     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Mar 18 '24 02:03 codecov[bot]

@bsbds Your PR is in conflict and cannot be merged.

mergify[bot] avatar Apr 18 '24 06:04 mergify[bot]

@bsbds Convert your pr to draft since CI failed

mergify[bot] avatar Apr 22 '24 07:04 mergify[bot]

@Mergifyio rebase

Phoenix500526 avatar Apr 25 '24 15:04 Phoenix500526

rebase

✅ Branch has been successfully rebased

mergify[bot] avatar Apr 25 '24 15:04 mergify[bot]

@bsbds Convert your pr to draft since CI failed

mergify[bot] avatar Apr 25 '24 15:04 mergify[bot]

@bsbds Convert your pr to draft since CI failed

mergify[bot] avatar Apr 26 '24 02:04 mergify[bot]

@bsbds Convert your pr to draft since CI failed

mergify[bot] avatar May 06 '24 01:05 mergify[bot]