[GitHub] [incubator-iceberg] rdblue commented on issue #141: Add case insensitive support to Parquet.

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

rdblue commented on issue #141: Add case insensitive support to Parquet.
URL: https://github.com/apache/incubator-iceberg/pull/141#issuecomment-480631648
 
 
   @xabriel, can you take another look? I've fixed the problem you found and validated there are tests that actually exercise case insensitive binding for the Parquet classes.
   
   To your question about how to reproduce this, you have to run a query that pushes filters down to Parquet. The filters you use can't be satisfied by partition filtering because Iceberg will drop those filters and won't push them down. I would recommend writing a dataset without partitioning and sorted by a categorical value, then test equality filters on that categorical value. That would hit this problem.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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