[FLINK-32616] FlinkStatement#executeQuery resource leaks when the input sql is not query

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.18.0
• Fix Version/s: 1.18.0
• Component/s: Table SQL / JDBC
• Labels: pull-request-available
• Assignee: FangYong
• Reporter: Shengkai Fang
• Created: 18/Jul/23 10:09
• Updated: 21/Jul/23 02:47
• Resolved: 21/Jul/23 02:47

Description

The current implementation just throw the exception if the input sql is not query. No one is responsible to close the StatementResult.

 

BTW, the current implementation just submit the sql to the gateway, which means the sql is executed. I just wonder do we need to expose this features to the users?

Issue Links

• links to: GitHub Pull Request #23027