codecov[bot]
commented
3 months ago 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
```diff @@ 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.
mergify[bot]
Phoenix500526
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)