kbendick commented on code in PR #4810:
URL: https://github.com/apache/iceberg/pull/4810#discussion_r883894158
##########
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:
We'd probably want to add the `HasIcebergCatalog` interface to other
catalogs as well, though that can be handled in a follow up once the interface
is in.
If / when we merge this, let's open an issue for adding `HasIcebergCatalog`
to other catalogs. I've got a newer way of tracking opened issues I've been
using so that's one way to ensure that it is at least not forgotten about =)
--
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]
