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

Na Li commented on SENTRY-1778:
-------------------------------

I don't think we have any query leak without this change. 

> Close query as soon as it is done
> ---------------------------------
>
>                 Key: SENTRY-1778
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1778
>             Project: Sentry
>          Issue Type: Improvement
>          Components: Sentry
>    Affects Versions: 1.8.0, sentry-ha-redesign
>            Reporter: Na Li
>            Assignee: Na Li
>            Priority: Minor
>              Labels: bite-sized, newbie
>
> Make sure we close all query results after we have finished with them. 
> Failure to do so will result in significant memory leaks in your application. 
>  mentioned in 
> http://www.datanucleus.org/products/accessplatform_4_1/jdo/performance_tuning.html
> Since we always close the transaction, and closing transaction will close the 
> query, the memory leak without this change should not be significant. This is 
> why this issue is not high priority. On the other hand, closing query result 
> sooner should not have side effect.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to