[
https://issues.apache.org/jira/browse/HIVE-7182?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14025327#comment-14025327
]
Swarnim Kulkarni commented on HIVE-7182:
----------------------------------------
Probably not immediately clear from the patch, but why the executeUpdate
statement was moved outside the if?
{code}
stmt.executeUpdate(createTable);
{code}
> ResultSet is not closed in JDBCStatsPublisher#init()
> ----------------------------------------------------
>
> Key: HIVE-7182
> URL: https://issues.apache.org/jira/browse/HIVE-7182
> Project: Hive
> Issue Type: Bug
> Reporter: Ted Yu
> Priority: Minor
> Attachments: HIVE-7182.patch
>
>
> {code}
> ResultSet rs = dbm.getTables(null, null,
> JDBCStatsUtils.getStatTableName(), null);
> boolean tblExists = rs.next();
> {code}
> rs is not closed upon return from init()
> If stmt.executeUpdate() throws exception, stmt.close() would be skipped - the
> close() call should be placed in finally block.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
