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

future possibilities and consequences #548

Merged
merged 2 commits into from
Jun 17, 2024
Merged

Conversation

wacban
Copy link
Contributor

@wacban wacban commented Jun 14, 2024

No description provided.

@wacban wacban requested a review from a team as a code owner June 14, 2024 16:22
neps/nep-0509.md Outdated

### Negative

- n1
- The hardware requirements will increase. In particular the network bandwidth and memory requirements will be higher.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm wonder if we should mention potential decrease of hardware requirement (disk requirement?) as well in positive section

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 - more specifically, small-stake nodes won't have to store state and apply chunks for all shards as they currently do.

- n1
- The hardware requirements will increase. In particular the network bandwidth and memory requirements will be higher.
- Additional limits will be put on the size of transactions, receipts and, more generally, cross shard communication.
- The dependency on cloud state sync will increase the centralization of the blockchain. This will be resolved separately by the decentralized state sync.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we specify what it starts having a dependency on state sync?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, it's mentioned above on explanation of validator roles.

@wacban
Copy link
Contributor Author

wacban commented Jun 17, 2024

@walnut-the-cat and @Longarithm I added the suggestion and I'm going to merge it now. Feel free to add any more comments and suggestions and I'll just do it in a follow up PR.

@wacban wacban merged commit 6069805 into state-validation Jun 17, 2024
1 check passed
@wacban wacban deleted the wacban-patch-1 branch June 17, 2024 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: APPROVED FIXES
Development

Successfully merging this pull request may close these issues.

3 participants