adam-christian-software commented on code in PR #3000:
URL: https://github.com/apache/polaris/pull/3000#discussion_r2515233417
##########
site/content/in-dev/unreleased/polaris-spark-client.md:
##########
@@ -22,17 +22,17 @@ type: docs
weight: 650
---
-Apache Polaris now provides Catalog support for Generic Tables (non-Iceberg
tables), please check out
-the [Polaris Catalog OpenAPI Spec]({{% ref
"polaris-api-specs/polaris-catalog-api.md" %}}) for Generic Table API specs.
+Polaris provides a Spark client to manage non-Iceberg tables through [Generic
Tables]({{% ref "generic-table.md" %}}).
-Along with the Generic Table Catalog support, Polaris is also releasing a
Spark client, which helps to
-provide an end-to-end solution for Apache Spark to manage Delta tables using
Polaris.
+{{< alert note >}}
+The Spark client can manage Iceberg tables and non-Iceberg tables.
-Note the Polaris Spark client is able to handle both Iceberg and Delta tables,
not just Delta.
+Users who only use Iceberg tables can use Spark without this client.
Review Comment:
Updated!
##########
site/content/in-dev/unreleased/polaris-spark-client.md:
##########
@@ -119,11 +119,16 @@ If you would like to use a version of the Spark client
that is currently not yet
build a Spark client jar locally from source. Please check out the Polaris
repo and refer to the Spark plugin
[README](https://github.com/apache/polaris/blob/main/plugins/spark/README.md)
for detailed instructions.
-## Limitations
-The Polaris Spark client has the following functionality limitations:
-1) Create table as select (CTAS) is not supported for Delta tables. As a
result, the `saveAsTable` method of `Dataframe`
+## Known Limitations
Review Comment:
Updated!
--
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]
