Ngone51 commented on pull request #28850: URL: https://github.com/apache/spark/pull/28850#issuecomment-657375831
yea, I think I have to agree with @srowen . The approach is not perfect as I thought previously, sorry for the misleading @wankunde Soft reference could probably work in this case, e.g. `activeThreadSession = new InheritableThreadLocal[SoftReference[SparkSession]]`, but it may also break something potentially with soft reference? Not sure about that. The use case is really abnormal. Maybe, you should just change your way. ---------------------------------------------------------------- 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. 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
