kumarUjjawal commented on PR #24227:
URL: https://github.com/apache/datafusion/pull/24227#issuecomment-5551465444
> im a little confused by what you mean by this
ParquetOptions is a public struct and is not #[non_exhaustive], so
downstream crates can construct it with an exhaustive ParquetOptions { ... }
literal. After this PR adds enable_rle_to_dictionary, those existing literals
fail to compile with E0063: missing field enable_rle_to_dictionary.
The docs/source/library-user-guide/upgrading/56.0.0.md should mention that
affected callers must add enable_rle_to_dictionary: false or migrate to
constructing from ParquetOptions::default().
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
