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:[EMAIL PROTECTED] Sent: Monday, July 16, 2007 9:22 PM To: [email protected] 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.
