[GitHub] [iceberg] wypoon commented on pull request #6056: Parquet: Remove the row position since parquet row group has it natively

[GitBox <gi...@apache.org>]

wypoon commented on PR #6056:
URL: https://github.com/apache/iceberg/pull/6056#issuecomment-1292619990

   So this change needs to wait until there is a parquet-mr release with a fix for [PARQUET-2161](https://issues.apache.org/jira/browse/PARQUET-2161), right?
   Just so I understand, for our usage, the bug manifests when `Parquet.ReadBuilder#split` is called (and thus when `Parquet.ReadBuilder#build` is called, the `VectorizedParquetReader` or `ParquetReader` is constructed with a `ParquetReadOptions` that has a `ParquetMetadataConverter.RangeMetadataFilter`), right? Why doesn't it affect the test I added in `TestSparkReaderDeletes`?


-- 
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: issues-unsubscribe@iceberg.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org