[GitHub] [arrow] amol- edited a comment on pull request #11591: ARROW-12163: [Java] Make compression levels configurable.

[GitBox <gi...@apache.org>]

amol- edited a comment on pull request #11591:
URL: https://github.com/apache/arrow/pull/11591#issuecomment-959234041


   > One more note. The JIRA linked is about adjusting the compression level used by compression algorithms (trading off speed for output size). I didnt notice a change here for that apologies if i missed it.
   
   @davisusanibar that seems true, the original goal of the JIRA ticket was to allow to configure the compression by specifying options like the level. Similarly to what we allow in C++ and Python where you can set `compression` and `compression_level` ( https://arrow.apache.org/docs/python/generated/pyarrow.Codec.html?highlight=codec#pyarrow.Codec ).
   
   It seems that the `CompressionCodec` and `CommonsCompressionFactory` do not allow to specify anything else than the `codecType` but not a level for such codecs. 
   
   Should `Lz4CompressionCodec` and `ZstdCompressionCodec` grow a constructor argument to specify the level?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org