MonkeyCanCode commented on code in PR #2957:
URL: https://github.com/apache/polaris/pull/2957#discussion_r2486582302


##########
plugins/spark/v3.5/getting-started/notebooks/SparkPolaris.ipynb:
##########
@@ -265,7 +265,6 @@
     "from pyspark.sql import SparkSession\n",
     "\n",
     "spark = (SparkSession.builder\n",
-    "  .config(\"spark.jars\", 
\"../polaris_libs/polaris-spark-3.5_2.12-1.2.0-incubating-SNAPSHOT-bundle.jar\")
  # TODO: add a way to automatically discover the Jar\n",

Review Comment:
   This jar is needed but we don't need to manually specify it. This is because 
I am copying the jar into the spark libs directory. So it will get auto loaded. 
A lot cleaner imo for getting start notebook so we don't need to update this 
line every time we bump the release number. 



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