jbonofre commented on code in PR #11478: URL: https://github.com/apache/iceberg/pull/11478#discussion_r1830477007
########## core/src/main/java/org/apache/iceberg/TableMetadata.java: ########## @@ -90,6 +90,8 @@ private static Map<String, String> persistedProperties(Map<String, String> rawPr persistedProperties.put( TableProperties.PARQUET_COMPRESSION, TableProperties.PARQUET_COMPRESSION_DEFAULT_SINCE_1_4_0); + persistedProperties.put( Review Comment: By adding this, we need to update a few tests (like Hive), as we see now `"write.delete.granularity"="file"` in the metadata persistence. I propose to update the tests. Maybe worth to add a test to check how to read "previous" files which don't contain the property ? -- 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