Skip to content

Allow to use IO uring instead of epoll - #3595

Merged
merlimat merged 4 commits into
apache:masterfrom
merlimat:io-uring
Nov 2, 2022
Merged

Allow to use IO uring instead of epoll#3595
merlimat merged 4 commits into
apache:masterfrom
merlimat:io-uring

Conversation

@merlimat

Copy link
Copy Markdown
Contributor

Motivation

Allow users to enable IO_uring for BK network transport.

Since the Netty implementation is still considered experimental, it will not be enabled by default, rather only when the -Denable.io_uring=true is passed.

@merlimat merlimat added this to the 4.16.0 milestone Oct 31, 2022

@hangc0276 hangc0276 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good job!

Comment thread bookkeeper-common/pom.xml Outdated
<dependency>
<groupId>io.netty.incubator</groupId>
<artifactId>netty-incubator-transport-native-io_uring</artifactId>
<version>${netty-iouring.version}</version>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove the version?

@hangc0276

Copy link
Copy Markdown
Contributor

@merlimat Would you please take a look at the failed tests?

@merlimat
merlimat merged commit b82e224 into apache:master Nov 2, 2022
@merlimat
merlimat deleted the io-uring branch November 2, 2022 17:31
yaalsn pushed a commit to yaalsn/bookkeeper that referenced this pull request Jan 30, 2023
* Allow to use IO uring instead of epoll

* Removed unused import

* Fixed license file

* Fixed license file
Ghatage pushed a commit to sijie/bookkeeper that referenced this pull request Jul 12, 2024
* Allow to use IO uring instead of epoll

* Removed unused import

* Fixed license file

* Fixed license file
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.

5 participants