metamodel-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (METAMODEL-1093) JDBC CompiledQuery ResultSets not closed properly
Date Thu, 09 Jun 2016 16:11:21 GMT

    [ https://issues.apache.org/jira/browse/METAMODEL-1093?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15322776#comment-15322776
] 

ASF GitHub Bot commented on METAMODEL-1093:
-------------------------------------------

Github user LosD commented on the issue:

    https://github.com/apache/metamodel/pull/107
  
    Just checked. It (and Postgres) is there. 
    
    A danger may be memory use. 


> JDBC CompiledQuery ResultSets not closed properly
> -------------------------------------------------
>
>                 Key: METAMODEL-1093
>                 URL: https://issues.apache.org/jira/browse/METAMODEL-1093
>             Project: Apache MetaModel
>          Issue Type: Bug
>            Reporter: Dennis Du Krøger
>            Assignee: Dennis Du Krøger
>
> When running compiled queries, the JdbcResultSet only returns the lease on the query,
it doesn't close the result set.
> For most JDBC drivers, that isn't really an issue, as they can easily handle connections
with multiple open result sets. However, the MySQL JDBC sometimes uses streaming result sets
(to save memory), of which only one can be open at a time.
> I almost have a PR ready, but I'm having a hard time making any kind of test that makes
sense: When I make a simple test, the driver seems to close the resultset automatically right
after execution, but that is not what we see in DataCleaner, and I'm not really sure why.
> I think I'll post it anyway, as it is rather large (I also tried to avoid repeats by
making sure that the JdbcDataContext only handles the connection, and the rest is handled
by try-with-resources as far as possible), so an early review would be nice.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message