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

treewide: Upgrade tokio to 1.32 #422

Closed
wants to merge 2 commits into from

Conversation

glittershark
Copy link
Member

Upgrade the version of tokio we depend on to version 1.32, to get the
version with tokio-rs/tokio#5925, my fix for a
performance issue in tokio::sync::broadcast. We use this to notify
workers when channels are removed from the channel coordinator, so we
want this fix to improve the performance of that process.

@glittershark glittershark changed the base branch from main to I989a0f7b49d01d130ab0586da90c0546f4a62b61 August 24, 2023 14:48
@readysetbot readysetbot force-pushed the Id45707e0f95ca0a76ea69952ea23e8c65f846983 branch from bc230a3 to 190f943 Compare August 24, 2023 14:48
@glittershark glittershark changed the base branch from I989a0f7b49d01d130ab0586da90c0546f4a62b61 to main August 24, 2023 14:53
@readysetbot readysetbot force-pushed the Id45707e0f95ca0a76ea69952ea23e8c65f846983 branch from 190f943 to 99ddf06 Compare August 24, 2023 14:53
Fix (or configure around) all existing clj-kondo lints, and run
clj-kondo in CI. Since it was unused, this deletes the test
file (generated by `lein new`) entirely. If we want to do unit tests
later we can add it back.

Fixes: REA-3371
Change-Id: I0db591510f6692bbb5bd161073afad4d3ad0a2c3
Reviewed-on: https://gerrit.readyset.name/c/readyset/+/5830
Tested-by: Buildkite CI
Reviewed-by: Ethan Donowitz <ethan@readyset.io>
Upgrade the version of tokio we depend on to version 1.32, to get the
version with tokio-rs/tokio#5925, my fix for a
performance issue in `tokio::sync::broadcast`. We use this to notify
workers when channels are removed from the channel coordinator, so we
want this fix to improve the performance of that process.

Change-Id: Id45707e0f95ca0a76ea69952ea23e8c65f846983
@readysetbot readysetbot force-pushed the Id45707e0f95ca0a76ea69952ea23e8c65f846983 branch from 99ddf06 to f83eda7 Compare August 24, 2023 18:57
@readysetbot
Copy link

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.

None yet

2 participants