RussellSpitzer commented on code in PR #4810:
URL: https://github.com/apache/iceberg/pull/4810#discussion_r885790871
##########
spark/v3.2/spark/src/main/java/org/apache/iceberg/spark/Spark3Util.java:
##########
@@ -648,6 +650,21 @@ public static org.apache.iceberg.Table
loadIcebergTable(SparkSession spark, Stri
return toIcebergTable(sparkTable);
}
+ /**
+ * Returns the underlying Iceberg Catalog object represented by a Spark
Catalog
+ * @param spark SparkSession used for looking up catalog reference
+ * @param catalogName The name of the Spark Catalog being referenced
+ * @return the Iceberg catalog class being wrapped by the Spark Catalog
+ */
+ public static Catalog loadIcebergCatalog(SparkSession spark, String
catalogName) {
Review Comment:
One issue with that is this is a Spark Table Catalog implementing class in
the current iteration, should we change that?
--
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]
