[ 
https://issues.apache.org/jira/browse/GOBBLIN-1786?focusedWorklogId=848512&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-848512
 ]

ASF GitHub Bot logged work on GOBBLIN-1786:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 02/Mar/23 04:49
            Start Date: 02/Mar/23 04:49
    Worklog Time Spent: 10m 
      Work Description: meethngala commented on code in PR #3643:
URL: https://github.com/apache/gobblin/pull/3643#discussion_r1122594517


##########
gobblin-data-management/src/main/java/org/apache/gobblin/data/management/copy/iceberg/IcebergDatasetFinder.java:
##########
@@ -92,6 +103,18 @@ public Iterator<IcebergDataset> getDatasetsIterator() 
throws IOException {
 
   protected IcebergDataset createIcebergDataset(String dbName, String tblName, 
IcebergCatalog icebergCatalog, Properties properties, FileSystem fs) {
     IcebergTable icebergTable = icebergCatalog.openTable(dbName, tblName);
-    return new IcebergDataset(dbName, tblName, icebergTable, properties, fs);
+    return new IcebergDataset(dbName, tblName, icebergTable, 
icebergCatalog.getCatalogUri(), properties, fs);

Review Comment:
   makes sense... I have associated the catalog Uri with `IcebergTable` in my 
latest commit!





Issue Time Tracking
-------------------

    Worklog Id:     (was: 848512)
    Time Spent: 6h 10m  (was: 6h)

> Support Other Catalog Types for Iceberg Distcp
> ----------------------------------------------
>
>                 Key: GOBBLIN-1786
>                 URL: https://issues.apache.org/jira/browse/GOBBLIN-1786
>             Project: Apache Gobblin
>          Issue Type: Improvement
>            Reporter: Meeth Gala
>            Priority: Major
>          Time Spent: 6h 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to