pitrou commented on code in PR #15194:
URL: https://github.com/apache/arrow/pull/15194#discussion_r1072188571
##########
cpp/src/arrow/ipc/options.h:
##########
@@ -67,6 +67,15 @@ struct ARROW_EXPORT IpcWriteOptions {
/// May only be UNCOMPRESSED, LZ4_FRAME and ZSTD.
std::shared_ptr<util::Codec> codec;
+ /// @brief Always use compression, regardless of the resulting output size
+ ///
+ /// If false, compression will be skipped for body buffers where the
expected compressed
+ /// size matches/exceeds the uncompressed size. Otherwise, compression will
be applied
+ /// indiscriminately. This option is ignored if no codec was supplied.
+ ///
+ /// This is true by default due to backwards-compatibility concerns.
+ bool compress_always = true;
Review Comment:
It would save decoder time. But also, if using a sophisticated encoding
strategy as mentioned above, it would also save encoder time.
--
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]
