gh-yzou commented on code in PR #1857:
URL: https://github.com/apache/polaris/pull/1857#discussion_r2150938022


##########
site/content/in-dev/unreleased/polaris-spark-client.md:
##########
@@ -128,3 +128,21 @@ The Polaris Spark client has the following functionality 
limitations:
 3) Rename a Delta table is not supported.
 4) ALTER TABLE ... SET LOCATION is not supported for DELTA table.
 5) For other non-Iceberg tables like csv, it is not supported.
+
+## Iceberg Spark Client compatibility with Polaris Spark Client
+The Polaris Spark client today is not designed to be used with Iceberg Spark 
client together. In other words,

Review Comment:
   @adutra since we have tested and clarified that we already support both 
approach today without this change. i updated the PR title as an improvement to 
reuse the shadowJar for maven publish.
   The published jar name is called 
polaris-spark-3.5_2.12-0.11.0-beta-incubating-SNAPSHOT-bundle.jar, which i 
think is a pretty clear name. If we prefer name like polaris-spark-bundle, i 
will need to create a separate project to just pack the uber jar, and I can do 
it in a follow up PR.
   



-- 
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]

Reply via email to