dnskr opened a new pull request #35234:
URL: https://github.com/apache/spark/pull/35234


   ### What changes were proposed in this pull request?
   The memory leak of `ExecutionListenerBus` was fixed in 
https://github.com/apache/spark/commit/4d90c5dc0efcf77ef6735000ee7016428c57077b 
by automatically cleaning it up when the `SparkSession` is GC'ed.
   This is a similar fix for the memory leak of `SQLAppStatusListener` that 
occurs when a new `SessionState` is touched.  The memory leak is automatically 
cleaned up when the `SharedState` is GC'ed.
   
   
   ### Why are the changes needed?
   The memory leak causes OOM in spark applications like Spark Thrift Server 
that creates many sessions.
   
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   
   ### How was this patch tested?
   A test has been added to the PR
   


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

Reply via email to