lifulong commented on code in PR #47281: URL: https://github.com/apache/spark/pull/47281#discussion_r1673481996
########## sql/core/src/main/scala/org/apache/spark/sql/SparkSession.scala: ########## @@ -113,6 +113,8 @@ class SparkSession private( private[sql] val sessionUUID: String = UUID.randomUUID.toString + private[sql] val jobArtifactState: JobArtifactState = JobArtifactState(sessionUUID, None) Review Comment: The change of this pr is try to reuse JobArtifactState in normal Spark sql query. set active jobArtifactState in each spark session make sure get right session uuid in method JobArtifactSet.getActiveOrDefault, then get isolated session aritifact set of each session. In kyuubi group share level, multi user may load their own udf jar, and may exist same udf class name in different jar, lead the first load class take effect always -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
