[FLINK-31398] Don't wrap with TemporaryClassLoaderContext in OperationExecutor

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.18.0, 1.17.1
• Component/s: Connectors / Hive, Table SQL / Client
• Labels: pull-request-available
• Assignee: Weijie Guo
• Reporter: luoyuxia
• Created: 10/Mar/23 12:09
• Updated: 21/Apr/23 17:18
• Resolved: 21/Apr/23 17:18

Description

Currently, method OperationExecutor#executeStatement in sql client will wrap currently with `

sessionContext.getSessionState().resourceManager.getUserClassLoader()`. Actually, it's not necessary. What' worse, 

it'll will cause the exception 'Trying to access closed classloader. Please check if you store xxx'  after quiting sql client. 

The reason is in `ShutdownHookManager`, it will register a hook after jvm shutdown. In `ShutdownHookManager`, it will

create `Configuration`. It will then access `Thread.currentThread().getContextClassLoader()` which is FlinkUserClassLoader, the FlinkUserClassLoader has been closed before. So, it'll then cause `'Trying to access closed classloader` exception.

 

 

Issue Links

• fixes: FLINK-31629 Trying to access closed classloader when submit query to restSqlGateway via SqlClient
• links to: GitHub Pull Request #22294