deniskuzZ commented on PR #11216:
URL: https://github.com/apache/iceberg/pull/11216#issuecomment-2580006220

   > @deniskuzZ: Thanks for testing out. We are working on Internal 
parquet/Avro/orc readers and writes. partition stats will use them. So, we 
don't need to go through these converters. I will retest all the data types 
once I use internal writers for partition stats.
   > 
   > PRs: #11919 #11904
   
   JFYI, once I removed `IdentityPartitionConverters.convertConstant` from the 
`statsToRecords->convertPartitionValues` issue with `partitionToPath` method is 
gone. 
   IDK, maybe you had some other idea of how to do the partition filtering on 
the returned stats, but I used the following 
   ````
   spec.partitionToPath(stats.unwrap().partition())
     .equals(partish.getPartition().getName())
   ````
   


-- 
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-unsubscr...@iceberg.apache.org

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


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

Reply via email to