[GitHub] [arrow] wjones127 commented on pull request #35455: GH-34375: [C++][Parquet] Ignore page header stats when page index enabled

["wjones127 (via GitHub)" <gi...@apache.org>]

wjones127 commented on PR #35455:
URL: https://github.com/apache/arrow/pull/35455#issuecomment-1542799456

   > Yes, by default page index is disabled. I am not sure if we should turn on it by default someday to align with what parquet-mr does.
   
   I think we eventually want to turn it on by default. We just release support for it in 12.0.0, but it's not yet used for any data skipping in the Parquet reader or dataset scanner (right?). I think we should add support for leveraging it in the readers in 13.0.0, and then if it seems to be working for users we can turn in on by default in 14.0.0. Does that seem like a reasonable plan?
   


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org