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

[fastx client] Added a function to sample authorities by stake #171

Merged
merged 1 commit into from
Jan 16, 2022

Conversation

gdanezis
Copy link
Collaborator

Sample an authority from the committee by weight according to its stake.

@gdanezis
Copy link
Collaborator Author

Hey @patrickkuo is this what you need to sample by authority? I am also happy to make a 'shuffle' function but thought I should check first.

@gdanezis gdanezis marked this pull request as draft January 13, 2022 22:33
@patrickkuo
Copy link
Contributor

does higher stake = higher trust? If yes then this will be useful for quick client data sync.
Should we also allow client to define an authorities whitelist which they will automatically trust/ have higher weight in the sample?

@gdanezis
Copy link
Collaborator Author

The authority weight is determined by the committee structure that in turn will be determined by the amount of stake that holders of tokens delegate to authorities. The assumption is that up to 1/3 of stake is byzantine. But that is not a subjective measure, so its global.

@gdanezis gdanezis marked this pull request as ready for review January 15, 2022 00:29
Copy link
Contributor

@patrickkuo patrickkuo left a comment

Choose a reason for hiding this comment

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

LGTM!

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.

[fastx distsys] implement authorities sampling according to stake distribution.
2 participants