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

Qos prioritizing stake #22556

Closed

Conversation

tao-stones
Copy link
Contributor

@tao-stones tao-stones commented Jan 18, 2022

Problem

Validator with x% stake should have x% block space

Summary of Changes

  • Add stake weight stage before sigverify (sakridge's Add tx weighting stage #21953)
  • Add additional metrics to track sender details per block, allowing to compare sender's stake against its transactions cost in a block. Ideally sender stake percentage would closely corresponding to its TXs percentage in a block.
  • Add logic to leader to stake weight shuffle unprocessed buffer leader prioritizes transactions with sender stake weight #22419
  • Drop lower weight batch to make room for new one (commit 32ec0d4)

Fixes #
Bot attack

@tao-stones tao-stones force-pushed the qos-prioritizing-stake branch 2 times, most recently from f64ace0 to 32ec0d4 Compare January 21, 2022 14:27
@tao-stones
Copy link
Contributor Author

gcp cluster test yields perfect stake percentage and transaction packing percentage matching:
image

@tao-stones
Copy link
Contributor Author

Make a new PR #22958 on tip of master with only necessary commits cherry-picked. Close this one

@tao-stones tao-stones closed this Feb 5, 2022
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.

2 participants