Indhumathi27 commented on PR #5649: URL: https://github.com/apache/hive/pull/5649#issuecomment-2754699316
@deniskuzZ Adding Shutdown hook on SessionState.Start actually creating a memory overhead. while testing the change in a cluster, noticed considerably lot of shutdown hooks. And also, the shutdown hook tried closing a session which is already closed. To overcome these issues, added a single ShutdownHook for the List of session states which are left unclosed. Handled this changes in the second commit of this PR. Please help to review. Thanks -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org
