Do not delete BrotliDecoder and let GraalVM DCE do it for us

[gsmet]

No description provided.

[zakkak]

Please don't merge this before https://github.com/graalvm/mandrel/actions/runs/897038267 completes (testing it against Mandrel 21.1.0).

[zakkak]

CI passed, so that's OK to be merged.

I am not sure I understand why we "We need to delete this class" though. Is this to prevent users from trying to use brotli due to some known issue with native-image?

[gsmet]

No it's because it doesn't work right now (it uses some native library and work is needed to make it work).

[cescoffier]

@zakkak should be fine no?

[zakkak]

No it's because it doesn't work right now (it uses some native library and work is needed to make it work).

In that case it would be nice to have a GH issue and link to it in the comment so that we can better track it.

@zakkak should be fine no?

Yup it's fine!

[gsmet]

Frankly, except if someone from the community is complaining and wants to implement it, I don't think any of us will have the time nor the willingness to look into it. That's why I haven't created an issue.