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

io_uring integration via new transport API #5320

Merged
merged 1 commit into from
Oct 18, 2024
Merged

Conversation

vietj
Copy link
Member

@vietj vietj commented Sep 14, 2024

Motivation

io_uring is not anymore incubating and part of Netty 4.2, we should support it in vertx core.

Changes

Add an API for transport configuration through Vert.x builder in addition of the existing prefer native option. In addition add io_uring to the transports as well as CI support.

The transport service SPI has been removed.

@vietj vietj added this to the 5.0.0 milestone Sep 14, 2024
@vietj vietj force-pushed the io_uring_integration branch 29 times, most recently from 1297a01 to 72bbe35 Compare September 16, 2024 12:55
@vietj vietj force-pushed the io_uring_integration branch 3 times, most recently from 8e11275 to 545910e Compare September 17, 2024 17:13
@franz1981
Copy link
Contributor

franz1981 commented Oct 1, 2024

Transport:supportFileRegion seems fine - I cannot see the failures you mentioned: was an old thing? @vietj

@vietj vietj self-assigned this Oct 6, 2024
@vietj
Copy link
Member Author

vietj commented Oct 6, 2024

@vietj vietj force-pushed the io_uring_integration branch 3 times, most recently from 795c40b to 2482fbd Compare October 18, 2024 08:36
Motivation:

io_uring is not anymore incubating and part of Netty 4.2, we should support it in vertx core.

Changes:

Add an API for transport configuration through Vert.x builder in addition of the existing prefer native option. In addition add io_uring to the transports as well as CI support.

The transport service SPI has been removed.
@vietj vietj marked this pull request as ready for review October 18, 2024 15:23
@vietj vietj merged commit ba82495 into master Oct 18, 2024
8 checks passed
@vietj vietj deleted the io_uring_integration branch October 18, 2024 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants