flow-go
flow-go copied to clipboard
[State Sync] Experiment with using pebble as the execution datastore db
Close #6017
Context
In this pull request:
- Added a CLI flag that controls which db type to use.
- Added skeleton of
StorageDBinterface andbadgerandpebbleimplementations. - Refactored execution data
trackerand execution datapruningmechanism.
Codecov Report
Attention: Patch coverage is 8.60215% with 170 lines in your changes missing coverage. Please review.
Project coverage is 41.42%. Comparing base (
60e14fd) to head (406a64a).
Additional details and impacted files
@@ Coverage Diff @@
## master #6180 +/- ##
==========================================
- Coverage 41.49% 41.42% -0.07%
==========================================
Files 2003 2008 +5
Lines 142757 142903 +146
==========================================
- Hits 59232 59196 -36
- Misses 77352 77532 +180
- Partials 6173 6175 +2
| Flag | Coverage Δ | |
|---|---|---|
| unittests | 41.42% <8.60%> (-0.07%) |
:arrow_down: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I created a fork of ipfs/go-ds-pebble with your changes here: https://github.com/onflow/go-ds-pebble. please update the replace to this repo, then I'll approve
I created a fork of
ipfs/go-ds-pebblewith your changes here: https://github.com/onflow/go-ds-pebble. please update the replace to this repo, then I'll approve
Thank you, @peter, for creating the forked repo. I have updated the replace.