Skip to content

Initialize DnsNameResolverBuilder at runtime for native images#14376

Merged
chrisvest merged 1 commit intonetty:4.1from
idelpivnitskiy:native
Sep 27, 2024
Merged

Initialize DnsNameResolverBuilder at runtime for native images#14376
chrisvest merged 1 commit intonetty:4.1from
idelpivnitskiy:native

Conversation

@idelpivnitskiy
Copy link
Copy Markdown
Member

Motivation:

After #14375 Netty/Testsuite/NativeImage/Client started to fail with: No instances of java.net.Inet4Address are allowed in the image heap as this class should be initialized at image runtime.

Modifications:

  • Add DnsNameResolverBuilder to native-image.properties;

Result:

Netty/Testsuite/NativeImage/Client is expected to start working again.

Motivation:

After netty#14375 Netty/Testsuite/NativeImage/Client started to fail with:
No instances of `java.net.Inet4Address` are allowed in the image heap
as this class should be initialized at image runtime.

Modifications:

- Add `DnsNameResolverBuilder` to `native-image.properties`;

Result:

Netty/Testsuite/NativeImage/Client is expected to start working again.
Copy link
Copy Markdown
Member

@chrisvest chrisvest left a comment

Choose a reason for hiding this comment

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

If the Graal build is happy, I'm happy.

@chrisvest chrisvest merged commit e87ce47 into netty:4.1 Sep 27, 2024
chrisvest pushed a commit that referenced this pull request Sep 27, 2024
)

Motivation:

After #14375 Netty/Testsuite/NativeImage/Client started to fail with: No
instances of `java.net.Inet4Address` are allowed in the image heap as
this class should be initialized at image runtime.

Modifications:

- Add `DnsNameResolverBuilder` to `native-image.properties`;

Result:

Netty/Testsuite/NativeImage/Client is expected to start working again.
@idelpivnitskiy idelpivnitskiy deleted the native branch September 28, 2024 21:38
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