[
https://issues.apache.org/jira/browse/TRB-101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17340169#comment-17340169
]
Thomas Vandahl commented on TRB-101:
------------------------------------
The group lifecycle is supposed to be handled by the pull tool lifecycle. After
the request is finished, all groups should have been returned to the pool. This
is what IntakeTool.dispose() does.
I cannot see currently what keeps this from happening. This will require deeper
research.
> Object leak in IntakeTool
> -------------------------
>
> Key: TRB-101
> URL: https://issues.apache.org/jira/browse/TRB-101
> Project: Turbine
> Issue Type: Bug
> Components: Core
> Affects Versions: Core 4.0 Final, Core 5.0
> Environment: Linux
> Reporter: Susi Berrington
> Priority: Major
>
> We're trying to upgrade to Turbine4.0 and are finding that everything works
> fine for a while but then seizes up (blocks).
> Turns out that there's an object leak in the init() method in IntakeTool.
> [http://svn.apache.org/viewvc/turbine/core/trunk/src/java/org/apache/turbine/services/intake/IntakeTool.java?revision=1886259&view=markup#l134]
> {code:java}
> List<Group> foundGroups = intakeService.getGroup(groupNames[i])
> .getObjects(pp);{code}
> getGroup() is doing borrowObject() on a commons-pool2 pool but when we do
> getObjects() our reference to the object borrowed by getGroup() is lost, we
> can never release the borrowed object.
> I couldn't believe that this was new to 4.0 so I reverted to 2.3.3. This
> code is the same but the object pool is 1.5.6 and the limit is not enforced
> so we get away with the number of objects just going up and up.
> A fix could be:
> {code:java}
> for (int i = groupNames.length - 1; i >= 0; i--)
> {
> Group foundGroup = null;
> try
> {
> foundGroup = intakeService.getGroup(groupNames[i]);
> List<Group> foundGroups = foundGroup.getObjects(pp);
> if (foundGroups != null)
> {
> for (Group group : foundGroups)
> {
> groups.put(group.getObjectKey(), group);
> }
> }
> }
> catch (IntakeException e)
> {
> log.error(e);
> } finally {
> if (foundGroup != null)
> {
> try
> {
> intakeService.releaseGroup(foundGroup);
> } catch (IntakeException intakeException) {
> log.error(intakeException, intakeException);
> }
> }
> }
> }
> {code}
>
> I'm trying to find a way to hack round this. I've created a class which
> extends IntakeTool and just implements the init() method but I can't
> initialise pullMap so whenever we tried to actually use Intake we get a
> NullPointerException as pullMap is still null but is assumed to be
> initialised, which is fair enough! Can think of a better work around?
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
