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

Robert Munteanu commented on OAK-3301:
--------------------------------------

I'm not sure about the static stuff. JUnit creates a new instance for each test 
run anyway, so statics should only be used for once-per-test-class logic, which 
I don't think is the case here. 

I can either make my own change use statics or make the other fields static ( 
which I slightly prefer ).

Thoughts?

> AbstractRepositoryUpgrade leaks Repository instances
> ----------------------------------------------------
>
>                 Key: OAK-3301
>                 URL: https://issues.apache.org/jira/browse/OAK-3301
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: upgrade
>            Reporter: Robert Munteanu
>             Fix For: 1.4
>
>         Attachments: 
> 0001-OAK-3301-AbstractRepositoryUpgrade-leaks-Repository-.patch
>
>
> AbstractRepositoryUpgradeTest creates (JCR) repository instances but does not 
> close them ( actually creates a repository each time a session is requested ).
> This leads to out of memory errors when the process limit is hit on a machine.



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

Reply via email to