Open bsbds opened 3 months ago
Attention: Patch coverage is 23.44828%
with 111 lines
in your changes missing coverage. Please review.
Project coverage is 75.39%. Comparing base (
e35b35a
) to head (80e3aeb
). Report is 121 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
crates/engine/src/rocksdb_engine/transaction.rs | 20.74% | 107 Missing :warning: |
crates/engine/src/memory_engine/transaction.rs | 0.00% | 4 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@bsbds Your PR is in conflict and cannot be merged.
@bsbds Convert your pr to draft since CI failed
@Mergifyio rebase
rebase
@bsbds Convert your pr to draft since CI failed
@bsbds Convert your pr to draft since CI failed
@bsbds Convert your pr to draft since CI failed
@bsbds Convert your pr to draft since CI failed
Please squash the commits
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)