Blazer-007 commented on code in PR #4058:
URL: https://github.com/apache/gobblin/pull/4058#discussion_r1811402899


##########
gobblin-data-management/src/test/java/org/apache/gobblin/data/management/copy/iceberg/IcebergTableTest.java:
##########
@@ -200,7 +212,8 @@ public void testNewTablePropertiesAreRegistered() throws 
Exception {
     TableIdentifier destTableId = TableIdentifier.of(dbName, "destTable");

Review Comment:
   Have fixed them properly, it was a fault on my side to entirely create new 
table while just modifying existing table to be created with partitionSpec 
fixed it smoothly.



-- 
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: dev-unsubscr...@gobblin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to