Xline feat: implement memory transaction

feat: implement memory 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 memory 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)

Mar 18 '24 02:03 bsbds

Codecov Report

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

Project coverage is 75.32%. Comparing base (e35b35a) to head (fe77c41). Report is 48 commits behind head on master.

Files	Patch %	Lines
crates/engine/src/memory_engine/transaction.rs	13.40%	82 Missing and 2 partials :warning:
crates/engine/src/memory_engine/mod.rs	90.00%	0 Missing and 1 partial :warning:

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #702      +/-   ##
==========================================
- Coverage   75.55%   75.32%   -0.24%     
==========================================
  Files         180      187       +7     
  Lines       26938    27665     +727     
  Branches    26938    27665     +727     
==========================================
+ Hits        20353    20838     +485     
- Misses       5366     5542     +176     
- Partials     1219     1285      +66

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