Skip to content

Support Unix Domain Sockets with io_uring transport#6026

Merged
tsegismont merged 1 commit into
eclipse-vertx:masterfrom
tsegismont:io_uring-uds
May 13, 2026
Merged

Support Unix Domain Sockets with io_uring transport#6026
tsegismont merged 1 commit into
eclipse-vertx:masterfrom
tsegismont:io_uring-uds

Conversation

@tsegismont
Copy link
Copy Markdown
Member

In Netty 4.2, Unix Domain Sockets with io_uring are supported.

But they were not integrated by Vert.x yet.

@tsegismont tsegismont added this to the 5.1.0 milestone Mar 18, 2026
@tsegismont tsegismont requested a review from vietj March 18, 2026 10:42
@tsegismont
Copy link
Copy Markdown
Member Author

@vietj please take a look at this draft.

In the long run, do you want to have a job for native io_uring + domain sockets, as we have for native epoll?

Here it's convenient because the job for native io_uring is temporarily disabled, so at least we have some smoke tests for TCP and HTTP client / servers.

@vietj
Copy link
Copy Markdown
Member

vietj commented Mar 18, 2026

I'd like to have things tests as they should be

@tsegismont
Copy link
Copy Markdown
Member Author

Can you elaborate?

@vietj
Copy link
Copy Markdown
Member

vietj commented Mar 18, 2026

@tsegismont do we have io uring fully passing on CI ?

@vietj
Copy link
Copy Markdown
Member

vietj commented Mar 18, 2026

I can see now only a few tests are executed in the new io uring profile

@tsegismont
Copy link
Copy Markdown
Member Author

@tsegismont do we have io uring fully passing on CI ?
I can see now only a few tests are executed in the new io uring profile

Not yet. This is why this is a draft PR that we will be ready for review when the NativeIoUring CI job can be enabled again.

In the meantime, I was wondering if, eventually, we'd need the NativeIoUring+DomainSockets job like we have for epoll

@vietj
Copy link
Copy Markdown
Member

vietj commented Mar 18, 2026

I don't know yet, I need to see how io uring testing comes

@tsegismont tsegismont closed this Mar 19, 2026
@tsegismont tsegismont deleted the io_uring-uds branch March 19, 2026 17:31
@tsegismont tsegismont restored the io_uring-uds branch March 19, 2026 17:32
@tsegismont tsegismont reopened this Mar 19, 2026
@tsegismont tsegismont marked this pull request as ready for review May 11, 2026 15:32
@vietj
Copy link
Copy Markdown
Member

vietj commented May 12, 2026

great, but for now I'd like to stick to smoke tests in the io_uring test profile

@tsegismont tsegismont removed the request for review from vietj May 12, 2026 09:04
In Netty 4.2, Unix Domain Sockets with io_uring are supported.

But they were not integrated by Vert.x yet.

Signed-off-by: Thomas Segismont <tsegismont@gmail.com>
@tsegismont
Copy link
Copy Markdown
Member Author

Merging as the failures are unrelated

@tsegismont tsegismont merged commit bbcaea2 into eclipse-vertx:master May 13, 2026
2 of 8 checks passed
@tsegismont tsegismont deleted the io_uring-uds branch May 13, 2026 16:07
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