The only thing we do on the server side is a ClassLoaderDelegate.

At Service Container:

private void registerClassLoader() throws Exception
 {
 ClassLoader cl = getClass().getClassLoader();
 mbeanServer.registerMBean(new ClassLoaderJMXWrapper(cl), CLASS_LOADER_OBJECT_NAME);
 }

This is the URL for the ServiceContainer:

http://anonsvn.jboss.org/repos/messaging/branches/Branch_Stable/tests/src/org/jboss/test/messaging/tools/container/ServiceContainer.java

ClassLoader.defineclass is a protected method. Do you make any sort of hacks to make the call on classLoader? (Reflection with accessible=true maybe?)

Usually the method that needs to be delegated is loadClass only. Maybe you're calling something that is not being delegated due to some "hacking" behavior added by AOP?

This is interesting:


If I changed ClassLoaderJMSWrapper to look on itself besides the delegate, everything works (even with the latest AOP):

public Class<?> loadClass(String name) throws ClassNotFoundException
 {
 try
 {
 if (name.endsWith("[]"))
 {
 name = name.substring(0, name.length() - 2);

 //The classloader of an array type is the classloader of it's element (if non primitive)

 Class cl = delegate.loadClass(name);

 Object arr = Array.newInstance(cl, 0);

 return arr.getClass();
 }
 else
 {
 return delegate.loadClass(name);
 }
 }
 catch (ClassNotFoundException e)
 {
 return super.loadClass(name);
 }
 }

"clebert.suconic@jboss.com" wrote:

Can you check if you are aways using the correct classloader? This might turn a bug on ClassLoaderJMXWrapper only. We just need to double check.

If instead of re-inventing the wheel, you used the DelegatingClassLoader from
JBoss Common Core, you'd already have the correct fix.

http://viewvc.jboss.org/cgi-bin/viewvc.cgi/common/common-core/trunk/src/main/java/org/jboss/util/loading/DelegatingClassLoader.java?revision=2787&view=markup

The super.loadClass() works because it invokes findLoadedClass().

And thanks a lot Flavia... I will make few more tests but this seems to fix the problem.

Flavia wrote:
I'm curious to know why we hadn't seen this error before.

Anyway... something changed on the interaction with the classLoader, and that hit the bug.


I'm changing it to use the DelegateClassLoader as Adrian suggested and it worked. So we are fine now!

We will need an AOP release anyway.

When I changed build-messaging.xml to use the default aopc I got another ClassNotFoundException, but everything passed when I built aop from SVN and replaced thirdpary, as Flavia fixed another problem.