flyrain commented on code in PR #4810:
URL: https://github.com/apache/iceberg/pull/4810#discussion_r876329175
##########
spark/v3.2/spark/src/main/java/org/apache/iceberg/spark/Spark3Util.java:
##########
@@ -648,6 +650,22 @@ 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) {
+ CatalogPlugin catalogPlugin =
spark.sessionState().catalogManager().catalog(catalogName);
+ Preconditions.checkArgument(catalogPlugin instanceof HasIcebergCatalog,
+ String.format("Cannot load Iceberg catalog from catalog %s because it
is not a Spark Iceberg catalog. Actual " +
Review Comment:
because it is not a Spark Iceberg catalog -> because it doesn't contain an
Iceberg catalog?
--
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]
