Skip to content

pack: exempt votes and bundles from pacing, schedule votes first #4682

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

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

Conversation

ptaffet-jump
Copy link
Contributor

The policy is now:

Bank 0 when pacing says it should be inactive (the first X ms of the block):

  1. If there's a vote, schedule it and return.
  2. If there's a bundle (almost certainly schedulable), schedule it and return.

Bank 0 when pacing says it should be active (the rest of the block):

  1. If there's a vote, schedule it and return.
  2. If there's a bundle that can be scheduled, schedule it and return.
  3. If there weren't any bundles, or the next one doesn't fit in the rest of the block, try to schedule a normal transaction
    Note: if there was a bundle but it couldn't be scheduled because of account conflicts, don't schedule anything.

Bank >0 when pacing says it should be inactive:

  1. If there's a vote, schedule it and return

Bank >0 when pacing says it should be active:

  1. If there's a vote, schedule it and return
  2. Try to schedule a normal transaction

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.

1 participant