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

[R4R] parity warp-like state sync #568

Merged
merged 4 commits into from
May 28, 2019
Merged

[R4R] parity warp-like state sync #568

merged 4 commits into from
May 28, 2019

Conversation

ackratos
Copy link
Contributor

@ackratos ackratos commented Apr 17, 2019

Description

node: #568
tendermint: bnb-chain/bnc-tendermint#68
cosmos: bnb-chain/bnc-cosmos-sdk#130
iavl: bnb-chain/bnc-tendermint-iavl#6

Rationale

https://docs.google.com/document/d/1rJ3FDa2wUht-NIrvAsNG8HkZ5iyGbmCxwyiLazIz3H0/edit?usp=sharing
tendermint/tendermint#828

Example

play with testnet with:
config.zip

Changes

Notable changes:

  • add each change in a bullet point here
  • ...

Preflight checks

  • build passed (make build)
  • tests passed (make test)
  • integration tests passed (make integration_test)
  • manual transaction test passed (cli invoke)

Already reviewed by

...

Related issues

... reference related issue #'s here ...

@HaoyangLiu
Copy link
Contributor

As the application data is growing bigger and bigger, the snapshot process which tiggered by breathblock will cost more and more time. Suppose this circumstance, the snapshot process is started, but before its completion, the node is restarted by some unkown reason, will the uncompleted snapshot process be resumed?

app/app.go Show resolved Hide resolved
@ackratos
Copy link
Contributor Author

ackratos commented May 7, 2019

As the application data is growing bigger and bigger, the snapshot process which tiggered by breathblock will cost more and more time. Suppose this circumstance, the snapshot process is started, but before its completion, the node is restarted by some unkown reason, will the uncompleted snapshot process be resumed?

no :( in such case we have to run snapshot command manually

@ackratos ackratos force-pushed the state_sync_warp branch 2 times, most recently from 6fc2fa3 to 65d99bd Compare May 8, 2019 08:35
@ackratos ackratos changed the title [WIP] parity warp-like state sync [R4R] parity warp-like state sync May 9, 2019
@ackratos ackratos self-assigned this May 9, 2019
HaoyangLiu
HaoyangLiu previously approved these changes May 9, 2019
HaoyangLiu
HaoyangLiu previously approved these changes May 9, 2019
@ackratos ackratos changed the base branch from master to develop May 27, 2019 07:43
Gopkg.lock Show resolved Hide resolved
@yutianwu yutianwu merged commit e64fee4 into develop May 28, 2019
@unclezoro unclezoro deleted the state_sync_warp branch May 10, 2022 06:10
forcodedancing pushed a commit that referenced this pull request May 19, 2022
* parity warp-like state sync

* fix unit-test

* revise a bit to support sync from any breathe block and code clean up

* update dependency
forcodedancing pushed a commit that referenced this pull request May 19, 2022
* parity warp-like state sync

* fix unit-test

* revise a bit to support sync from any breathe block and code clean up

* update dependency
forcodedancing pushed a commit that referenced this pull request May 19, 2022
* parity warp-like state sync

* fix unit-test

* revise a bit to support sync from any breathe block and code clean up

* update dependency
forcodedancing pushed a commit that referenced this pull request May 19, 2022
* parity warp-like state sync

* fix unit-test

* revise a bit to support sync from any breathe block and code clean up

* update dependency
forcodedancing pushed a commit that referenced this pull request May 19, 2022
* parity warp-like state sync

* fix unit-test

* revise a bit to support sync from any breathe block and code clean up

* update dependency
forcodedancing pushed a commit that referenced this pull request May 19, 2022
* parity warp-like state sync

* fix unit-test

* revise a bit to support sync from any breathe block and code clean up

* update dependency
forcodedancing pushed a commit that referenced this pull request May 19, 2022
* parity warp-like state sync

* fix unit-test

* revise a bit to support sync from any breathe block and code clean up

* update dependency
forcodedancing pushed a commit that referenced this pull request May 19, 2022
* parity warp-like state sync

* fix unit-test

* revise a bit to support sync from any breathe block and code clean up

* update dependency
forcodedancing pushed a commit that referenced this pull request May 19, 2022
* parity warp-like state sync

* fix unit-test

* revise a bit to support sync from any breathe block and code clean up

* update dependency
forcodedancing pushed a commit that referenced this pull request May 19, 2022
* parity warp-like state sync

* fix unit-test

* revise a bit to support sync from any breathe block and code clean up

* update dependency
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants