Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: snapshot and pruning functionality #140

Merged
merged 41 commits into from
Mar 24, 2022
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
41 commits
Select commit Hold shift + click to select a range
f348dc1
refactor snapshot config
p0mvn Mar 13, 2022
9b34767
create pruning package and move pruning options into it
p0mvn Mar 13, 2022
f45b027
introduce snapshot options struct, similar to pruning options
p0mvn Mar 13, 2022
1468df4
implement pruning manager and unit test
p0mvn Mar 15, 2022
8e37e0d
snapshot inteval must be equal to pruning keep recent
p0mvn Mar 15, 2022
8ad567e
multistore PruneHeight stub and unit test with mocks
p0mvn Mar 15, 2022
49b269c
implement the barebones logic for pruning keep-every after snapshot a…
p0mvn Mar 16, 2022
e6be907
rename multistore PruneHeight to PruneSnapshotHeight and implement it
p0mvn Mar 16, 2022
4a78172
add logs
p0mvn Mar 16, 2022
19831f8
fix unit tests
p0mvn Mar 16, 2022
fa53501
refactor baseapp_test.go
p0mvn Mar 16, 2022
de9c030
refactor pruning options.go
p0mvn Mar 16, 2022
4df3ffd
finalize TestSnapshotWithPruning and ensure all unit tests pass
p0mvn Mar 16, 2022
ed59a43
implement GetSnapshotBlockRetentionHeights
p0mvn Mar 16, 2022
02b4c03
add pruning types and constructors to sdk types package
p0mvn Mar 16, 2022
b658d9b
clean up pruning and snaphot error messages and names
p0mvn Mar 16, 2022
e318a90
remove dependency on pruning types from baseapp_test.go
p0mvn Mar 16, 2022
1a78b81
further clean up baseapp_test.go
p0mvn Mar 16, 2022
fff4055
improve comment in options.go
p0mvn Mar 16, 2022
4487e79
fix comment in pruning/manager.go
p0mvn Mar 16, 2022
6e70a75
remove junk file
p0mvn Mar 16, 2022
24478ee
remove pruning-keep-every and use snapshot-interval instead
p0mvn Mar 17, 2022
d0ceed6
remveo all references to pruning-keep-every
p0mvn Mar 17, 2022
8733ac7
allow snapshot-keep-recent of 0, add more unit tests
p0mvn Mar 17, 2022
0b62de4
bug fixes related to prune-everything
p0mvn Mar 17, 2022
9d34297
add unit tests for app.init(), remove validation from snapshot manage…
p0mvn Mar 17, 2022
835cf32
clean up
p0mvn Mar 17, 2022
157bb38
optimize deletion from list while iterating
p0mvn Mar 17, 2022
e8e94cf
golangci-lint run --fix
p0mvn Mar 17, 2022
183b8e6
Update store/types/store.go
p0mvn Mar 18, 2022
3775b3e
restore config tests
p0mvn Mar 18, 2022
db8661c
rename Type to PruningStrategy, prefix strategy names with "Pruning"
p0mvn Mar 18, 2022
3be7f3d
change Snapshot to SnapshotIfApplicable
p0mvn Mar 21, 2022
c9d827c
Update snapshots/manager.go
p0mvn Mar 21, 2022
197552f
change log level in SnapshotIfApplicable
p0mvn Mar 21, 2022
948387d
add comments for stores in snapshot manager
p0mvn Mar 21, 2022
4f48a83
check for snapshots enabled with non-multi store
p0mvn Mar 22, 2022
56b4781
add documentation in snapshots/README.md
p0mvn Mar 22, 2022
bdb1112
get latest version directly from the db in Snapshot() to avoid data race
p0mvn Mar 22, 2022
a619355
update documentation in pruning/README.md and snapshots/README.md
p0mvn Mar 22, 2022
bc941a0
update iavl to v6 with the data race fix
p0mvn Mar 22, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
update iavl to v6 with the data race fix
  • Loading branch information
p0mvn committed Mar 22, 2022
commit bc941a08f79c3cad008da4dd028ed4b698c0fefc
2 changes: 1 addition & 1 deletion go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -69,4 +69,4 @@ replace github.com/gin-gonic/gin => github.com/gin-gonic/gin v1.7.0

replace github.com/tendermint/tm-db => github.com/osmosis-labs/tm-db v0.6.5-0.20210911033928-ba9154613417

replace github.com/cosmos/iavl => github.com/osmosis-labs/iavl v0.17.3-osmo-v5
replace github.com/cosmos/iavl => github.com/osmosis-labs/iavl v0.17.3-osmo-v6
2 changes: 2 additions & 0 deletions go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -617,6 +617,8 @@ github.com/ory/dockertest v3.3.5+incompatible h1:iLLK6SQwIhcbrG783Dghaaa3WPzGc+4
github.com/ory/dockertest v3.3.5+incompatible/go.mod h1:1vX4m9wsvi00u5bseYwXaSnhNrne+V0E6LAcBILJdPs=
github.com/osmosis-labs/iavl v0.17.3-osmo-v5 h1:B6Saw2AK0StgfSSgi2rUc5B8MDa1vV1nGJUyYM7THus=
github.com/osmosis-labs/iavl v0.17.3-osmo-v5/go.mod h1:lJEOIlsd3sVO0JDyXWIXa9/Ur5FBscP26zJx0KxHjto=
github.com/osmosis-labs/iavl v0.17.3-osmo-v6 h1:xM/ESKmbyS7hx6Ye4xSitLbofSVw+/j3MhGTZr4aFxE=
github.com/osmosis-labs/iavl v0.17.3-osmo-v6/go.mod h1:lJEOIlsd3sVO0JDyXWIXa9/Ur5FBscP26zJx0KxHjto=
github.com/osmosis-labs/tm-db v0.6.5-0.20210911033928-ba9154613417 h1:otchJDd2SjFWfs7Tse3ULblGcVWqMJ50BE02XCaqXOo=
github.com/osmosis-labs/tm-db v0.6.5-0.20210911033928-ba9154613417/go.mod h1:dptYhIpJ2M5kUuenLr+Yyf3zQOv1SgBZcl8/BmWlMBw=
github.com/otiai10/copy v1.6.0 h1:IinKAryFFuPONZ7cm6T6E2QX/vcJwSnlaA5lfoaXIiQ=
Expand Down