btcsuite / btcd / 6901316365

56%

master: 57%

Build Type

Pull #1688

github

Committed by kcalvinalvin

Commit Message

blockchain: better Ancestor with skiplists

On startup, Ancestor call was taking a lot of time when the node was
loading the blockindex onto memory. This change speeds up the Ancestor
function significantly and speeds up the node during startup.

On testnet3 at blockheight ~2,500,000, the startup was around 30seconds
on current main and was 5 seconds with this change. Below is a benchstat
result showing the significant speedup.

goos: darwin
goarch: arm64
pkg: github.com/utreexo/utreexod/blockchain
           │     old.txt      │               new.txt                │
           │      sec/op      │    sec/op     vs base                │
Ancestor-8   120819.301µ ± 5%   7.013µ ± 19%  -99.99% (p=0.000 n=10)

           │  old.txt   │            new.txt             │
           │    B/op    │    B/op     vs base            │
Ancestor-8   0.000 ± 0%   0.000 ± 0%  ~ (p=1.000 n=10) ¹
¹ all samples are equal

           │  old.txt   │            new.txt             │
           │ allocs/op  │ allocs/op   vs base            │
Ancestor-8   0.000 ± 0%   0.000 ± 0%  ~ (p=1.000 n=10) ¹
¹ all samples are equal

Pull Request Pull Request #1688: blockchain: Add ancestor optimization to finding Ancestor

Run Details

24 of 24 new or added lines in 1 file covered. (100.0%)

85 existing lines in 1 file now uncovered.

27975 of 49875 relevant lines covered (56.09%)

25202.92 hits per line