Merge pull request #22291 from karalabe/fix-pruner-compaction

core/state/pruner: fix compaction after pruning
This commit is contained in:
Péter Szilágyi 2021-02-08 19:19:38 +02:00 committed by GitHub
commit 944d901436
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -188,8 +188,15 @@ func prune(maindb ethdb.Database, stateBloom *stateBloom, middleStateRoots map[c
cstart := time.Now() cstart := time.Now()
for b := byte(0); b < byte(16); b++ { for b := byte(0); b < byte(16); b++ {
log.Info("Compacting database", "range", fmt.Sprintf("%#x-%#x", b, b+1), "elapsed", common.PrettyDuration(time.Since(cstart))) var (
if err := maindb.Compact([]byte{b}, []byte{b + 1}); err != nil { start = []byte{b << 4}
end = []byte{(b+1)<<4 - 1}
)
log.Info("Compacting database", "range", fmt.Sprintf("%#x-%#x", start, end), "elapsed", common.PrettyDuration(time.Since(cstart)))
if b == 15 {
end = nil
}
if err := maindb.Compact(start, end); err != nil {
log.Error("Database compaction failed", "error", err) log.Error("Database compaction failed", "error", err)
return err return err
} }
@ -229,7 +236,7 @@ func (p *Pruner) Prune(root common.Hash) error {
// Reject if the accumulated diff layers are less than 128. It // Reject if the accumulated diff layers are less than 128. It
// means in most of normal cases, there is no associated state // means in most of normal cases, there is no associated state
// with bottom-most diff layer. // with bottom-most diff layer.
return errors.New("the snapshot difflayers are less than 128") return fmt.Errorf("snapshot not old enough yet: need %d more blocks", 128-len(layers))
} }
// Use the bottom-most diff layer as the target // Use the bottom-most diff layer as the target
root = layers[len(layers)-1].Root() root = layers[len(layers)-1].Root()