Bug 780642 (SOA-3081) - Need BRMS-594 included in SOA Platform
Summary: Need BRMS-594 included in SOA Platform
Keywords:
Status: CLOSED NEXTRELEASE
Alias: SOA-3081
Product: JBoss Enterprise SOA Platform 5
Classification: JBoss
Component: JBoss Rules
Version: 5.1.0 GA
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: ---
Assignee: Default User
QA Contact:
URL: http://jira.jboss.org/jira/browse/SOA...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-05-19 16:24 UTC by Matt Davis
Modified: 2011-05-19 17:06 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-05-19 17:06:19 UTC
Type: Bug


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker SOA-3081 0 None Closed Need BRMS-594 included in SOA Platform 2012-05-07 01:35:01 UTC

Description Matt Davis 2011-05-19 16:24:30 UTC
Help Desk Ticket Reference: https://c.na7.visual.force.com/apex/Case_View?id=500A0000007AQwS&sfdc.override=1
project_key: SOA

Fix for BRMS-594 needs to be include in SOA platform

I am running Drools (actually JBoss Rules 5.1.0 ) in Glassfish.

My deployable is an EJB that uses a StatelessKnowledgeSession to run
some rules that I get from a KnowledgeAgent that has a Changeset with a
.drl in it.
Therefore, I have placed the drools compiler into the deployable and it
all appears to work.

Glassfish has a clever little mechanism that, when you unload a
deployable that had some open streams, it tells you all about it (and
kindly closes them).

Glassfish is reporting that InputStreams are being left open that were
opened in the isPackage() method of
org.drools.commons.jci.compilers.EclipseJavaCompiler.
That method has the following code in it:-

final InputStream is =
pClassLoader.getResourceAsStream(ClassUtils.convertClassToResourcePath(pClazzName));
if (is != null) { return false; }

So, when "is" is not null (i.e. open), it is forgotten about and not closed.

Isn't this a potential file handle leak?

Comment 1 Matt Davis 2011-05-19 16:25:10 UTC
Link: Added: This issue Cloned from JBRULES-2928


Comment 2 Matt Davis 2011-05-19 16:25:10 UTC
Link: Added: This issue Cloned from BRMS-594


Comment 3 Matt Davis 2011-05-19 17:06:19 UTC
Issue is fixed in Drools 5.2


Note You need to log in before you can comment on or make changes to this bug.