ajantha-bhat commented on code in PR #32:
URL: https://github.com/apache/polaris-tools/pull/32#discussion_r2407182075
##########
iceberg-catalog-migrator/cli/src/test/java/org/apache/polaris/iceberg/catalog/migrator/cli/AbstractCLIMigrationTest.java:
##########
@@ -116,12 +114,6 @@ protected void beforeEach() {
@AfterEach
protected void afterEach() {
- // manually refreshing catalog due to missing refresh in Nessie catalog
- // https://github.com/apache/iceberg/pull/6789
- // create table will call refresh internally.
- sourceCatalog.createTable(TableIdentifier.of(BAR, "tblx"),
schema).refresh();
Review Comment:
Since for polaris, source and target catalog uses same warehouse (Needed for
migration). This was failing in CI when target catalog was trying to create
table as the same path table exists.
Now. I removed this as the above mentioned Nessie issue (in the comment) is
fixed and we don't need this workaround.
--
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]
