[
https://issues.apache.org/jira/browse/GOBBLIN-1786?focusedWorklogId=847076&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-847076
]
ASF GitHub Bot logged work on GOBBLIN-1786:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 23/Feb/23 01:26
Start Date: 23/Feb/23 01:26
Worklog Time Spent: 10m
Work Description: ZihanLi58 commented on code in PR #3643:
URL: https://github.com/apache/gobblin/pull/3643#discussion_r1115162459
##########
gobblin-data-management/src/main/java/org/apache/gobblin/data/management/copy/iceberg/IcebergCatalog.java:
##########
@@ -17,10 +17,25 @@
package org.apache.gobblin.data.management.copy.iceberg;
+import org.apache.iceberg.catalog.Catalog;
+
/**
* Any catalog from which to access {@link IcebergTable}s.
*/
public interface IcebergCatalog {
IcebergTable openTable(String dbName, String tableName);
+ String getCatalogUri();
+
+ /**
+ * Adding a sub interface to help us provide an association between {@link
Catalog} and {@link IcebergCatalog}.
+ * This helps us resolve to the Catalog to its concrete implementation class
+ * Primarily needed to access `newTableOps` method which only certain {@link
Catalog} derived classes open for public access
+ */
+ interface CatalogSpecifier {
Review Comment:
If we are saying icebergCatalog's behavior is binding with the underlying
catalog type, given that we already have one easy way
(CatalogUtil.loadCatalog()) to load the underlying catalog easily, I think you
can just keep with the original implementation, where you just specify the
icebergCatalog type and then under icebergCatalog, you force the specific
catalog (hiveCatalog for example). On IcebergMetataWriter side, we can just use
CatalogUtil to load the catalog...
Issue Time Tracking
-------------------
Worklog Id: (was: 847076)
Time Spent: 4h (was: 3h 50m)
> 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: 4h
> Remaining Estimate: 0h
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
