xupefei commented on code in PR #48120:
URL: https://github.com/apache/spark/pull/48120#discussion_r1766999756


##########
repl/src/test/scala/org/apache/spark/repl/ReplSuite.scala:
##########
@@ -396,4 +396,41 @@ class ReplSuite extends SparkFunSuite {
     Main.sparkContext.stop()
     System.clearProperty("spark.driver.port")
   }
+
+  test("register artifacts via SparkSession.addArtifact") {

Review Comment:
   I added one more test, which defines a UDF that initialises an external 
class added as an artifact.
   
   > how does this work with a JobArtifactSet?
   
   Can you elaborate? Afaik `JobArtifactSet` is not involved here since it's 
the artifact path that is applied when an active SparkSession is applied.
   
   Classpath - It's the other way around: the session classpath is laid over 
the global one.



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

Reply via email to