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

Unpin tokio for non-rpc crates #26957

Merged

Conversation

CriesofCarrots
Copy link
Contributor

Problem

#25028 pinned tokio across all solana crates. However, in actuality only solana-rpc needs to be pinned to avoid the RPC stalls, and the pin is causing problems for downstream users of solana client crates.

Summary of Changes

Unpin tokio everywhere except solana-rpc

@CriesofCarrots
Copy link
Contributor Author

CriesofCarrots commented Aug 5, 2022

Incidentally, I confirmed that the stalls are still evident on current tokio (v1.20.1), that a bin with this PR does not exhibit the stalls, and that this PR resolves the issues for downstream users importing solana crates.

@CriesofCarrots CriesofCarrots merged commit 66919e5 into solana-labs:master Aug 6, 2022
mergify bot pushed a commit that referenced this pull request Aug 6, 2022
(cherry picked from commit 66919e5)

# Conflicts:
#	banks-server/Cargo.toml
#	client/Cargo.toml
#	ledger-tool/Cargo.toml
#	net-utils/Cargo.toml
#	storage-bigtable/Cargo.toml
#	test-validator/Cargo.toml
mergify bot pushed a commit that referenced this pull request Aug 6, 2022
mergify bot added a commit that referenced this pull request Aug 6, 2022
* Unpin tokio for non-rpc crates (#26957)

(cherry picked from commit 66919e5)

# Conflicts:
#	banks-server/Cargo.toml
#	client/Cargo.toml
#	ledger-tool/Cargo.toml
#	net-utils/Cargo.toml
#	storage-bigtable/Cargo.toml
#	test-validator/Cargo.toml

* Fix conflicts

Co-authored-by: Tyera Eulberg <[email protected]>
mergify bot added a commit that referenced this pull request Aug 6, 2022
Unpin tokio for non-rpc crates (#26957)

(cherry picked from commit 66919e5)

Co-authored-by: Tyera Eulberg <[email protected]>
apfitzge pushed a commit to apfitzge/agave that referenced this pull request Aug 9, 2022
xiangzhu70 pushed a commit to xiangzhu70/solana that referenced this pull request Aug 17, 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.

1 participant