pitrou commented on a change in pull request #8811:
URL: https://github.com/apache/arrow/pull/8811#discussion_r543206245
##########
File path: cpp/src/arrow/ipc/options.h
##########
@@ -65,6 +65,20 @@ struct ARROW_EXPORT IpcWriteOptions {
/// like compression
bool use_threads = true;
+ /// \brief Whether to emit dictionary deltas
+ ///
+ /// If false, a changed dictionary for a given field will emit a full
+ /// dictionary replacement.
+ /// If true, a changed dictionary will be compared against the previous
+ /// version. If possible, a dictionary delta will be omitted, otherwise
+ /// a full dictionary replacement.
+ ///
+ /// Default is false to maximize stream compatibility.
Review comment:
No, I simply meant that it's off for compatibility with implementations
that don't support delta dictionaries.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
