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

state-sync: enable snapshotting by default #7311

Closed
4 tasks
tac0turtle opened this issue Sep 15, 2020 · 4 comments
Closed
4 tasks

state-sync: enable snapshotting by default #7311

tac0turtle opened this issue Sep 15, 2020 · 4 comments
Labels
T: Client UX T: Dev UX UX for SDK developers (i.e. how to call our code) T:Docs Changes and features related to documentation. T: UX

Comments

@tac0turtle
Copy link
Member

tac0turtle commented Sep 15, 2020

Summary

Currently, snapshotting is defaulted to off. While testing stargate-2 I had to stop and start my running node in order to enable this setting, and I had to sync a second node because people were not aware they had to turn this feature on.

Problem Definition

State-sync will not be able to be used if not on by default

Proposal

Turn snapshotting on by default.

The second part would be to make it worth with the application pruning setting. It will be hard to inform people to keep these in sync.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@tac0turtle tac0turtle added T: UX T: Dev UX UX for SDK developers (i.e. how to call our code) T: Client UX sync T:Docs Changes and features related to documentation. labels Sep 15, 2020
@alexanderbez
Copy link
Contributor

#7166 (comment) This was intentional and you were part of that discussion. I think the idea was that we keep it disabled until it has been battle-tested.

@tac0turtle
Copy link
Member Author

Hmm will close this issue but how is it suppose to get battle tested if one needs to turn it on but no one knows this.

@alexanderbez
Copy link
Contributor

By documentation and marketing it. You don't do it by blindly enabling it in case it has issues.

@tac0turtle
Copy link
Member Author

Hopefully we can move forward by simple default. I just spent an hour or so debugging a users state sync setup only to find no nodes in the stargate testnet had snapshotting enabled.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T: Client UX T: Dev UX UX for SDK developers (i.e. how to call our code) T:Docs Changes and features related to documentation. T: UX
Projects
None yet
Development

No branches or pull requests

2 participants