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

[WIP] Add first draft of validator architecture #1

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@paulhauner
Member

paulhauner commented Nov 3, 2018

Here is my draft thoughts on how the validator architecture should work.

To do:

  • Improve "Reasoning" section. It's still scrappy.
  • Add some diagrams?

If you have the privileges, feel free to just push changes to this branch if they wont be contentious :)

@paulhauner paulhauner changed the title from [WIP] Add first draft of validator-architecture to [WIP] Add first draft of validator architecture Nov 3, 2018

@spble

Looks great!!

Only thing I see missing is the case where Vsched & Vwatch are in a separate binary all on their own.

I've just added some typo suggestions while I was reading through.

Show resolved Hide resolved validator-architecture.md Outdated
Show resolved Hide resolved validator-architecture.md Outdated
obtain up-to-date blockchain information. This condition might caused by either
node or network faults.
4. A beacon node should be able to outsource shard syncing and block production
to a remote server. This avoids placing an `O(c)` bound upon the beacon

This comment has been minimized.

@spble

spble Nov 3, 2018

Member

clarification of what c is?

Show resolved Hide resolved validator-architecture.md Outdated
Show resolved Hide resolved validator-architecture.md Outdated
Show resolved Hide resolved validator-architecture.md Outdated
Show resolved Hide resolved validator-architecture.md Outdated
Show resolved Hide resolved validator-architecture.md Outdated
Show resolved Hide resolved validator-architecture.md Outdated
Fixed a few small gramatical issues and typos in the validator-archit…
…ecture document, as per comments left on PR#1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment