lifulong commented on PR #47281: URL: https://github.com/apache/spark/pull/47281#issuecomment-2222487866
> If we want this to be done properly, I think we should refactor the whole thing. The thread local in `JobArtifactSet` has to live in SparkSession instead for example. `JobArtifactSet` is not thread local, i think you means `JobArtifactState`, it is thread local, `JobArtifactState` has been in SparkSession in this change pr i add a jobArtifactState variable in spark session , which type is `JobArtifactState` is this meet your expection? -- 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