kbendick opened a new pull request, #5377: URL: https://github.com/apache/iceberg/pull/5377
This PR stems from https://github.com/apache/iceberg/pull/5305 and covers just the `FunctionCatalog` ## FunctionCatalog This allows users of `SparkCatalog` and `SparkSessionCatalog` to use functions (such as the `iceberg_version` function added here) without having to register it as a UDF. All Iceberg functions that we register into the function catalog are accessible when used with an Iceberg spark catalog and: 1. No namespace is referenced - the storage partitioned joins implementation requires this. e.g. `my_catalog.iceberg_versio()`. **Note** - Using `iceberg_version` typically does not work, as Spark adds the namespace to the call. Hence the `system` namespace syntax should be preferred. 2. The `system` namespace is referenced, to match called procedure syntax. **Note** this only works right now with the `SparkCatalog`, as the `SparkSessionCatalog` has logic in Spark [to verify that the namespace exists](https://github.com/apache/iceberg/pull/5305#discussion_r927178316). e.g. `my_catalog.system.iceberg_version()` or `system.iceberg_version()` if already having called `USE my_catalog`. ### iceberg_version function This also adds a simple function `iceberg_version`, which simply returns the (short) version string. This is mostly for testing but will be useful on its own. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
