Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

TransactionScheduler: CLI and hookup for central-scheduler #33890

Merged
merged 4 commits into from
Nov 13, 2023

Conversation

apfitzge
Copy link
Contributor

Problem

Need a way to use the new BankingStage w/ central scheduler

Summary of Changes

  • Factor out some code for spawning legacy/thread-local-multi-iterator banking stage threads
  • Add BlockProductionMethod::CentralScheduler variant
  • Add path to spawn new scheduling thread, workers, and legacy voting threads
  • Re-use existing banking stage test to verify new production method works in basic fashion

Fixes #

@codecov
Copy link

codecov bot commented Oct 27, 2023

Codecov Report

Merging #33890 (d3b37ce) into master (ba112a0) will increase coverage by 0.0%.
Report is 4 commits behind head on master.
The diff coverage is 98.2%.

@@           Coverage Diff            @@
##           master   #33890    +/-   ##
========================================
  Coverage    81.8%    81.9%            
========================================
  Files         810      810            
  Lines      218148   218247    +99     
========================================
+ Hits       178650   178814   +164     
+ Misses      39498    39433    -65     

@apfitzge apfitzge marked this pull request as ready for review October 27, 2023 08:31
@apfitzge apfitzge requested a review from tao-stones October 27, 2023 08:31
Copy link
Contributor

@tao-stones tao-stones left a comment

Choose a reason for hiding this comment

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

lgtm

@apfitzge apfitzge merged commit 81a007b into solana-labs:master Nov 13, 2023
ASTRONAU2023 added a commit to ASTRONAU2023/solana that referenced this pull request Nov 13, 2023
TransactionScheduler: CLI and hookup for central-scheduler (solana-labs#33890)
@apfitzge apfitzge deleted the scheduler_hookup branch November 13, 2023 14:26
ASTRONAU2023 added a commit to ASTRONAU2023/solana that referenced this pull request Nov 13, 2023
Revert "TransactionScheduler: CLI and hookup for central-scheduler (solana-labs#33890)"
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants