Skip to content

Backport Netty frame handler to 5.x.x-stable #1665

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

Merged
merged 1 commit into from
Aug 4, 2025
Merged

Conversation

acogoluegnes
Copy link
Contributor

No description provided.

@acogoluegnes acogoluegnes changed the base branch from main to 5.x.x-stable August 4, 2025 15:02
This commit adds a Netty-based frame handler. All network IO are based
on Netty when this frame handler is in use. The client library can
benefit of all the flexibility and advantages of Netty: simple API,
pluggable IO layer (Java NIO by default, or e.g. native Epoll on Linux).

The Netty frame handler deprecates the internal one using Java NIO.

The blocking IO frame handler is still the default one in 5.x. The
Netty frame handler should be the only one in 6.x.

Fixes #1664

(cherry picked from commit 74f538d)

Conflicts:
	pom.xml
	src/main/java/com/rabbitmq/client/ConnectionFactory.java
	src/main/java/com/rabbitmq/client/impl/AMQConnection.java
	src/main/java/com/rabbitmq/client/impl/HeartbeatSender.java
	src/main/java/com/rabbitmq/client/impl/Utils.java
	src/test/java/com/rabbitmq/client/test/ClientTestSuite.java
	src/test/java/com/rabbitmq/client/test/TestUtils.java
	src/test/java/com/rabbitmq/client/test/functional/Heartbeat.java
	src/test/java/com/rabbitmq/client/test/ssl/SslContextFactoryTest.java
	src/test/java/com/rabbitmq/client/test/ssl/VerifiedConnection.java
@acogoluegnes acogoluegnes marked this pull request as ready for review August 4, 2025 15:51
@acogoluegnes acogoluegnes merged commit a1f7479 into 5.x.x-stable Aug 4, 2025
3 checks passed
@acogoluegnes acogoluegnes deleted the netty-5-x branch August 4, 2025 15:58
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.

1 participant