HyukjinKwon commented on code in PR #48906:
URL: https://github.com/apache/spark/pull/48906#discussion_r1851125378
##########
sql/core/src/main/scala/org/apache/spark/sql/SparkSession.scala:
##########
@@ -133,14 +133,34 @@ class SparkSession private(
/** Tag to mark all jobs owned by this session. */
private[sql] lazy val sessionJobTag = s"spark-session-$sessionUUID"
+ /**
+ * A UUID that is unique on the thread level. Used by managedJobTags to make
sure that a same
+ * tag from two threads does not overlap in the underlying
SparkContext/SQLExecution.
+ */
+ private[sql] lazy val threadUuid = new InheritableThreadLocal[String] {
Review Comment:
Could we have a simple test for this inheritance? Otherwise, looks fine from
a cursory look.
--
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]
