openjpa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pinaki Poddar" <ppod...@bea.com>
Subject RE: PCRegistry ClassLoader memory leak
Date Tue, 17 Jul 2007 02:26:40 GMT
Just to clarify:
I meant it should be the loader that loaded Person.class where Person is
the persistence capable class, *not*
org.apache.openjpa.enhance.PersistenceCapable.class.getClassLoader(). 

But testing in multi-classloader environment is required to validate any
changes of this nature.


Pinaki Poddar
972.834.2865

-----Original Message-----
From: Kevan Miller [mailto:kevan.miller@gmail.com] 
Sent: Monday, July 16, 2007 9:22 PM
To: dev@openjpa.apache.org
Subject: Re: PCRegistry ClassLoader memory leak


On Jul 16, 2007, at 6:43 PM, Pinaki Poddar wrote:

> Meta.fieldTypes is declared as Class[].
> They are accessed via PCRegistry::getFieldTypes(Class pcClass).
>
> The suggestion of "Change fieldTypes to be Strings instead" *perhaps* 
> meant to change Meta.fieldTypes to String[] (as field type names). And

> only during
> PCRegistry::getFieldTypes() 'lazily' converting the String[]s to 
> Class[]es (by the same ClassLoader of the PersistenceCapable class?).

Right, you'll need a ClassLoader to load from. Defaulting the the CL of
the PersistenceCapable class might work (for Geronimo), but doesn't seem
right either...

--kevan

Notice:  This email message, together with any attachments, may contain information  of  BEA
Systems,  Inc.,  its subsidiaries  and  affiliated entities,  that may be confidential,  proprietary,
 copyrighted  and/or legally privileged, and is intended solely for the use of the individual
or entity named in this message. If you are not the intended recipient, and have received
this message in error, please immediately return this by email and then delete it.

Mime
View raw message