wgtmac commented on code in PR #34873:
URL: https://github.com/apache/arrow/pull/34873#discussion_r1177224767
##########
java/vector/src/main/java/org/apache/arrow/vector/ipc/ArrowWriter.java:
##########
@@ -53,6 +53,8 @@ public abstract class ArrowWriter implements AutoCloseable {
protected static final Logger LOGGER =
LoggerFactory.getLogger(ArrowWriter.class);
+ protected static final int compressionLevel = 3;
Review Comment:
We can use an `Optional<int> level` parameter and pass it to specific codecs
via `createCodec`. `Lz4CompressionCodec` and `ZstdCompressionCodec` can respect
the value when it is set or use specific default level if value is unset.
BTW, I just checked that `FramedLZ4CompressorOutputStream` does not have the
concept of level. It requires user to fine-tune the parameters:
https://commons.apache.org/proper/commons-compress/apidocs/org/apache/commons/compress/compressors/lz4/FramedLZ4CompressorOutputStream.Parameters.html
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
