RussellSpitzer commented on code in PR #4560:
URL: https://github.com/apache/iceberg/pull/4560#discussion_r856571223
##########
spark/v3.2/spark/src/test/java/org/apache/iceberg/spark/source/TestMetadataTablesWithPartitionEvolution.java:
##########
@@ -212,7 +214,6 @@ public void testFilesMetadataTableFilter() throws
ParseException {
sql("CREATE TABLE %s (id bigint NOT NULL, category string, data string)
USING iceberg " +
"TBLPROPERTIES ('commit.manifest-merge.enabled' 'false')", tableName);
initTable();
-
Review Comment:
I think if we want to make this change we need to do it in all the tests and
currently this is missing format changes for testFilesMetadataTable and
testWithUnknownTransfer. Probably fine to just keep it as is and match in the
new tests
--
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]
