Skip to content

cyper-core: drop the Send bound on CompioExecutor - #74

Merged
Berrysoft merged 1 commit into
compio-rs:masterfrom
lvella:executor-no-send
Sep 14, 2026
Merged

Berrysoft merged 1 commit into
compio-rs:masterfrom
lvella:executor-no-send

Conversation

@lvella

@lvella lvella commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

compio::runtime::spawn does not need it, and hyper leaves the choice to the executor. Lets HTTP/2 stream tasks be !Send on the compio runtime.

compio::runtime::spawn does not need it, and hyper leaves the choice to
the executor. Lets HTTP/2 stream tasks be !Send on the compio runtime.

@Berrysoft Berrysoft left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Thanks!

@Berrysoft
Berrysoft merged commit 0e955e3 into compio-rs:master Sep 14, 2026
33 checks passed
Berrysoft pushed a commit that referenced this pull request Sep 14, 2026
compio::runtime::spawn does not need it, and hyper leaves the choice to
the executor. Lets HTTP/2 stream tasks be !Send on the compio runtime.
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.

2 participants