Github user LingHao commented on a diff in the pull request:
https://github.com/apache/metamodel/pull/187#discussion_r219397449
--- Diff: jdbc/src/main/java/org/apache/metamodel/jdbc/JdbcDataContext.java
---
@@ -513,6 +514,8 @@ protected DataSet executeQuery(Connection connection,
Query query, boolean close
// otherwise
close(connection);
throw e;
+ } finally {
+ FileHelper.safeClose(statement);
--- End diff --
Yes, you are right. I recognized this stupid mistake, so I closed the PR
soon. :-)
---
