cloud-fan commented on pull request #28128: URL: https://github.com/apache/spark/pull/28128#issuecomment-622286327
I'm not against adding a clear method to free a session, but I'd like to understand more about the memory leak. According to https://github.com/apache/spark/pull/24807#issuecomment-500871806 , if the memory leak is caused by the listener, can we simply register that listener only once? That listener just null out the `defaultSession` at the end of application, and I don't see why we need to add one listener per spark session. ---------------------------------------------------------------- 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