wgtmac opened a new pull request, #36955: URL: https://github.com/apache/arrow/pull/36955
### Rationale for this change Keep the behavior consistent with parquet-mr and arrow-rs. ### What changes are included in this PR? Slight breaking change in ColumnProperties to set default encoding to std::nullopt. Then let column writer to decide default encoding according to data page version and column type. ### Are these changes tested? Modified TestBooleanValuesWriter in column_writer_test.cc to check the default encodings. ### Are there any user-facing changes? Yes, default encoding of boolean type has been switched to RLE when data page version is v2. -- 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]
