Conversation
Codecov Report
@@ Coverage Diff @@
## master #216 +/- ##
==========================================
- Coverage 68.57% 68.51% -0.07%
==========================================
Files 27 27
Lines 2126 2128 +2
==========================================
Hits 1458 1458
- Misses 593 595 +2
Partials 75 75
|
creachadair
left a comment
There was a problem hiding this comment.
The change seems fine, but it would be helpful if the description gave a bit of context for why this is necessary.
|
yes, in the sdk and tendermint we Call delete on the db but with lam trees the space on disk is not reclaimed until compaction is completed, in our environment we write so often the db doesn't have enough time to sufficiently compact itself. By adding force compact I have two things planned, in the short term I wrote a tool that prunes blocks, state and app state then calls force compact on the entire db. With this tool, after pruning the stores I would need to compact it. In the mid term id like to add force compacting to the sdk to avoid seekcompaction on goleveldb. The overall goal is optimisations with this. It allows the application to control compaction instead of passing it on to the default settings of the db |
No description provided.