Native: Brotli4J compression requires explicit config about quarkus

[jedla97]

Just a note. This is also happening on baremetal. You can check it with provided reproducer by running mvn clean verify -Dnative

1. mvn install -Dnative
2. ./target/reproducer-with-quarkus-1.0.0-SNAPSHOT-runner
3. Go to http://127.0.0.1:8080/compression/text and see quarkus log

[Karm]

@jcarranzan

Here is a working example, tested with Quarkus 3.10.0 on Linux aarch64 (see README.md) and Linux amd64, using Mandrel native-image, Mandrel-23.1.3.1-Final (JDK 21.0.3+9-LTS).

See:
https://github.com/Karm/dev-null/tree/main/brotli4j-native

The example is heavily simplified version of your original reproducer, because the issue has nothing to do with either OpenShift or some other dependencies you had there.

You don't need to do anything in the Java code, you just need to mark Brotli4jLoader.java for runtime init and you need to specify resource config so as libbrotli.so gets bundled in the final executable file as a resource.

See application.properties.

[Karm]

Hello @jcarranzan,

I have your reproducer working properly in native. Let me polish the how-to and share it with you.

This is neither Quarkus nor GraalVM issue per se. It's just that AOT requires some more config work to get it working.
Quarkus extensions shield users from this complexity, but since there is none for this specific library usage (or none I know of), you need to add some manual configs.

[quarkus-bot]

/cc @Karm (mandrel), @galderz (mandrel), @geoand (openshift), @iocanel (openshift), @zakkak (mandrel,native-image)

[gsmet]

Is this usage widespread? Would it make sense to develop a small extension similar to what I did here: https://github.com/quarkiverse/quarkus-opencv/ ?

[Karm]

@gsmet https://quarkusio.zulipchat.com/#narrow/stream/187038-dev/topic/Webserver.20compression/near/438350071

I'd say yes. Unless I misunderstood and it's already a part of e.g. Vert.x.

[cescoffier]

I would not say 'widespread'. It's starting to be used. Vert.x already has the support (via netty), and due to some ugly substitution business, we have it on the classpath.

@Karm Based on your exploration, I would do the same thing I did for RocksDB, Snappy... If a built-time property enables Brotli, we include the appropriate native library into the native executable and mark Broli4Loarder as runtime init.

[melloware]

Should we re-open this ticket then?

[Karm]

@melloware @cescoffier

Should we re-open this ticket then?

I don't know. Not a bug... it's a missing feature :)

I will work on the seamless integration regardless of this particular issue being open or not.

[cescoffier]

I guess the next to be PR should be enough. This ticket provides the workaround (correct instructions).

[melloware]

@Karm thanks i will monitor for PR's!

[michalvavrik]

@Karm thanks i will monitor for PR's!

yeah, we also need to know to change our tests, so let's all monitor instead of having a tracker that will notify us