[GitHub] [incubator-iceberg] rdsr commented on issue #417: Adding support for time-based partitioning on long column type

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

rdsr commented on issue #417: Adding support for time-based partitioning on long column type
URL: https://github.com/apache/incubator-iceberg/issues/417#issuecomment-528452639
 
 
   Seems like for existing tables which rely on identity partitioning it may be best to not change it.  Both Spark and Pig seems to be reading identity partition value from the `DataFile` instead of from records, so it should work for both existing data written with Hive - which does not add the partition data in the file and new data/partitions written with Iceberg which does add partition data in the file.

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