wangyum commented on a change in pull request #31850:
URL: https://github.com/apache/spark/pull/31850#discussion_r595107710
##########
File path:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
##########
@@ -155,7 +155,24 @@ private[hive] class HiveClientImpl(
}
}
- ShutdownHookManager.addShutdownHook(() => state.close())
+ private def closeState(): Unit = {
+ // these temp files are registered in o.a.h.u.ShutdownHookManager too
+ // the state.close() will delete them if they are not null and try remove
them
+ // from the o.a.h.u.ShutdownHookManager which causes IllegalStateException
+ // we delete them ahead with a high priority hook here and set them to
null to
+ // bypass the deletion in state.close()
Review comment:
Capitalizes the first letter of the first word in a sentence?
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
