hyperxpro / Brotli4j

Brotli4j provides Brotli compression and decompression for Java.
Apache License 2.0
102 stars 35 forks source link

Make EncoderJNI and EncoderJNI.Wrapper public? #123

Closed deshanxiao closed 8 months ago

deshanxiao commented 8 months ago

Hello, I notice that DecoderJNI and DecoderJNI.Wrapper are marked as public class. Is it possible to mark EncoderJNI and EncoderJNI.Wrapper as public too?

This requirement comes from the fact that we want to support brotli codec in Apache ORC. However, Bytebuffer overflow needs to be considered in ORC, and frequent copies are not very friendly to performance. We want to use EncoderJNI directly to support these special scenario.

hyperxpro commented 8 months ago

Sure, feel free to do a PR.