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

Reply via email to