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.

Sign up for GitHub

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

Jump to bottom

Use ThreadLocalMultiIterator for tests #34947

Merged
merged 2 commits into from
Jan 25, 2024
Merged

Use ThreadLocalMultiIterator for tests #34947

merged 2 commits into from
Jan 25, 2024

Conversation

apfitzge
Copy link
Contributor

Problem

Summary of Changes

  • Use the MI method for block-production, which will do forwarding, while proper fixes in the client's used by tests are worked on.

Fixes #

@apfitzge apfitzge marked this pull request as ready for review January 25, 2024 17:04
CriesofCarrots
CriesofCarrots previously approved these changes Jan 25, 2024
View reviewed changes
@CriesofCarrots CriesofCarrots added the automerge Merge this Pull Request automatically once CI passes label Jan 25, 2024
@CriesofCarrots
Copy link
Contributor

I can't be bothered to wait for the coverage step to complete. Merging early

@CriesofCarrots CriesofCarrots merged commit 29737ab into solana-labs:master Jan 25, 2024
34 of 36 checks passed
@apfitzge apfitzge deleted the test-hackfix/use-mi-for-tests branch January 25, 2024 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CriesofCarrots CriesofCarrots CriesofCarrots approved these changes

Assignees
No one assigned
Labels
automerge Merge this Pull Request automatically once CI passes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@apfitzge @CriesofCarrots