hyperxpro/Brotli4j

Make EncoderJNI and EncoderJNI.Wrapper public?

deshanxiao opened this issue · 1 comments

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.

Sure, feel free to do a PR.