szehon-ho commented on code in PR #8299:
URL: https://github.com/apache/iceberg/pull/8299#discussion_r1296861141
##########
core/src/main/java/org/apache/iceberg/BaseMetastoreCatalog.java:
##########
@@ -149,6 +149,12 @@ public BaseMetastoreCatalogTableBuilder(TableIdentifier
identifier, Schema schem
this.identifier = identifier;
this.schema = schema;
this.tableProperties.putAll(tableDefaultProperties());
+
+ // Explicitly set default Parquet compression codecs for new tables
Review Comment:
Took a look, REST catalog table builder doesnt seem to build TableMetadata
themselves (gets from the response object). So this wont help that case. I
tried to organize the code a little bit into TableMetadata class in any case
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
