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

Create new netty-graal module for native image configuration #6657

Merged
merged 3 commits into from
Jan 24, 2022

Conversation

yawkat
Copy link
Member

@yawkat yawkat commented Dec 14, 2021

The graal configuration that netty needs was previously only present in the http-netty module (HttpNettyFeature). This patch creates a new netty-graal module for that configuration, so that other modules that use netty but not http-netty (in particular micronaut-grpc) can use the same configuration.

Resolves #6650

The graal configuration that netty needs was previously only present in the http-netty module (HttpNettyFeature). This patch creates a new netty-graal module for that configuration, so that other modules that use netty but not http-netty (in particular micronaut-grpc) can use the same configuration.

Resolves #6650
@yawkat yawkat added this to the 3.3.0 milestone Dec 14, 2021
Copy link
Contributor

@ilopmar ilopmar left a comment

Choose a reason for hiding this comment

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

I've tried this locally with the gRPC module and everything works as expected

@ilopmar
Copy link
Contributor

ilopmar commented Dec 17, 2021

I've created micronaut-projects/micronaut-grpc#437 in gRPC module with the changes to use this once we release Micronaut 3.3.0

@graemerocher
Copy link
Contributor

I am still unconvinced a new module needed. Especially if #6661 is merged in which case that module will be need to allow configurability of the Netty buffer allocator

@yawkat
Copy link
Member Author

yawkat commented Dec 29, 2021

Then #6661 should also be in the new module

@graemerocher
Copy link
Contributor

Why? it relates to configuring the buffer allocator and is in the netty-buffer module. What more appropriate module would it live in? Configuring the buffer allocator seems something that would be desirable in a gRPC app as well

@yawkat
Copy link
Member Author

yawkat commented Dec 30, 2021

imo the purpose of the netty-buffer module is to provide the netty implementation of io.micronaut.core.io.buffer.ByteBuffer, general netty configuration is unrelated

@yawkat yawkat marked this pull request as ready for review January 10, 2022 08:33
@yawkat
Copy link
Member Author

yawkat commented Jan 10, 2022

moved this to buffer-netty too, as per #6661

@yawkat yawkat linked an issue Jan 14, 2022 that may be closed by this pull request
@jameskleeh jameskleeh merged commit 6fd63f8 into 3.3.x Jan 24, 2022
@jameskleeh jameskleeh deleted the netty-feature branch January 24, 2022 19:18
yawkat added a commit that referenced this pull request Jun 24, 2022
* Create new netty-graal module for native image configuration
The graal configuration that netty needs was previously only present in the http-netty module (HttpNettyFeature). This patch creates a new netty-graal module for that configuration, so that other modules that use netty but not http-netty (in particular micronaut-grpc) can use the same configuration.

Resolves #6650

* checkstyle

* move changes to buffer-netty module
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.

Generalize HttpNettyFeature for other uses of netty
4 participants