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

feat: new leader schedule by vote delegation #4597

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jstarry
Copy link

@jstarry jstarry commented Jan 23, 2025

Problem

SIMD-0180 proposes a new change to the leader schedule algorithm to use vote account delegations rather than total node delegations.

Summary of Changes

  • Switch over to the new leader schedule algorithm using a feature gate. The new leader schedule algorithm will be applied for the epoch immediately following the epoch in which the feature gate was activated in.

  • A new constructor (fn new_keyed_by_vote_account) was added to the existing LeaderSchedule struct which has a vote delegation map as a parameter rather than a list of node ids and their accumulated stakes.

Fixes #
Feature Gate Issue: #4573

@jstarry jstarry force-pushed the feat/leader-schedule-migration branch from d32331b to 9f16cee Compare January 28, 2025 13:06
@joncinque
Copy link

This PR contains changes to the solana sdk, which will be moved to a new repo within the next week, when v2.2 is branched from master.

Please merge or close this PR as soon as possible, or re-create the sdk changes when the new repository is ready at https://github.com/anza-xyz/solana-sdk

@jstarry jstarry force-pushed the feat/leader-schedule-migration branch 8 times, most recently from 903ab77 to bbd8566 Compare February 6, 2025 10:33
@jstarry jstarry force-pushed the feat/leader-schedule-migration branch from bbd8566 to 4b7393e Compare February 6, 2025 10:39
@jstarry jstarry marked this pull request as ready for review February 6, 2025 10:41
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