search

cosmos / iavl

Merkleized IAVL+ Tree implementation in Go
Apache License 2.0
420 stars 263 forks source link

chore: backport lazy pruning to v1.x.x #906

Closed czarcas7ic closed 6 months ago

czarcas7ic commented 7 months ago

Should be merged to v1.1.x as well.

czarcas7ic commented 7 months ago

@cool-develope test failure looks concerning though, having trouble running locally as well, looking into it

czarcas7ic commented 7 months ago

@cool-develope I see you reran and it worked, is this non determinism expected / existed prior to this PR?

cool-develope commented 7 months ago

@cool-develope I see you reran and it worked, is this non determinism expected / existed prior to this PR?

yes, that one is non-deterministic which is related to random sets, I will fix it later

czarcas7ic commented 7 months ago

Great, I will test this on Osmosis then and report back

czarcas7ic commented 6 months ago

Closing in favor of https://github.com/cosmos/iavl/pull/911