wgtmac opened a new pull request, #35230:
URL: https://github.com/apache/arrow/pull/35230

   ### Rationale for this change
   
   Currently parquet writer only supports enabling page index for all columns. 
It would be good to enable/disable at the column level as sometimes it may not 
be useful for some columns but it pays to create them.
   
   ### What changes are included in this PR?
   
   Similar to 
`WriterProperties::Builder::enable_dictionary/disable_dictionary`, this patch 
adds 
`WriterProperties::Builder::enable_write_page_index/disable_write_page_index` 
and keep it backward compatible to enable/disable for all columns.
   
   ### Are these changes tested?
   
   Added `ParquetPageIndexRoundTripTest::EnablePerColumn` to cover the new 
settings.
   
   ### Are there any user-facing changes?
   
   Yes, users are now more flexible to enable/disable page index.


-- 
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]

Reply via email to