pitrou commented on code in PR #15194:
URL: https://github.com/apache/arrow/pull/15194#discussion_r1087952601
##########
cpp/src/arrow/ipc/options.h:
##########
@@ -67,6 +67,16 @@ struct ARROW_EXPORT IpcWriteOptions {
/// May only be UNCOMPRESSED, LZ4_FRAME and ZSTD.
std::shared_ptr<util::Codec> codec;
+ /// @brief Minimum space savings percentage required for compression to be
applied
+ ///
+ /// Space savings is calculated as (1.0 - compressed_size /
uncompressed_size).
+ ///
+ /// For example, if min_space_savings = 0.1, a 100-byte body buffer won't
undergo
+ /// compression if its expected compressed size exceeds 90 bytes. If this
option isn't
+ /// set, compression will be used indiscriminately. If no codec was
supplied, this
Review Comment:
```suggestion
/// compression if its expected compressed size exceeds 90 bytes. If this
option is
/// set to 0.0, compression will be used indiscriminately. If no codec was
supplied, this
```
--
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]
