|Index: jboss/src/main/org/jboss/ejb/ContainerFactory.java
|===================================================================
|RCS file:
|/products/cvs/ejboss/jboss/src/main/org/jboss/ejb/ContainerFactory.java,v
|retrieving revision 1.64
|diff -r1.64 ContainerFactory.java
|329c329
|<       ClassLoader cl = new URLClassLoader( jarUrls,
|Thread.currentThread().getContextClassLoader() );
|---
|>       ClassLoader cl = Thread.currentThread().getContextClassLoader() ;


Scott DONT YOU COMMIT THIS ATROCITY

thanks


marc

|
|-----Original Message-----
|From: Castro, David
|To: 'Scott M Stark'
|Cc: '[EMAIL PROTECTED]'
|Sent: 2/22/01 11:24 PM
|Subject: RE: [jBoss-Dev] CVS update: newsite/business jboss-tomcat.html
|
|Allright, I must be going nuts here.  In my deployment, tomcat-test is
|not running optimized under ANY circumstances.  The classloaders are
|behaving strangly.  I hope someone else will check this out and confirm
|or deny.
|
|> -----Original Message-----
|> From: Castro, David
|> Sent: Thursday, February 22, 2001 11:03 PM
|> To: Castro, David; 'Scott M Stark'
|> Cc: '[EMAIL PROTECTED]'
|> Subject: RE: [jBoss-Dev] CVS update: newsite/business
|> jboss-tomcat.html
|>
|>
|> Im going to contradict myself here.  Without
|> ContextClassLoaderInterceptor  youre never going to run
|> optimized.  I was kind of confused by the tomcat-test numbers
|> at first, but when I looked at the code it is clear - no
|> optimization without ContextClassLoaderInterceptor.
|>
|> Check me out here as I may be doing something else wrong that
|> is causing these problems.  I dont think so though.
|>
|> > -----Original Message-----
|> > From: Castro, David
|> > Sent: Thursday, February 22, 2001 10:40 PM
|> > To: 'Scott M Stark'
|> > Cc: '[EMAIL PROTECTED]'
|> > Subject: RE: [jBoss-Dev] CVS update: newsite/business
|> > jboss-tomcat.html
|> >
|> >
|> > I can confirm that Jdk12Interceptor does not get the job done
|> > for my application.
|> >
|> > I think the case where you need ContextClassLoaderInterceptor
|> > is when classes in the war depend directly on classes in
|> > shared libraries.  In JPS and tomcat-test, all required
|> > classes are physically in the war from what I can see.
|> >
|> > However, any call that has a parameter or return value whose
|> > class is loaded from an ejb-jar/war will not be optimized if
|> > you dont use the ContextClassLoaderInterceptor.  For example,
|> > running tomcat-test without it, the call to the remote
|> > interface is optimized because no interesting args or return
|> > vals.  The call to the home interface (.create()) is not
|> > optimized however because the remote interface that it
|> > returns cannot be loaded from a shared classloader.
|> >
|> > Therefore I strongly recommend that you put in
|> > ContextClassLoaderInterceptor if you want deployment and
|> > optimization to be consistent.
|> >
|> > Otherwise your instructions are immaculate.  I installed cvs
|> > on cygwin so I could follow them as written.  Very smooth.
|> >
|> > sorry about the delay in responding, I was busy for a couple
|> > of days there.
|> >
|> >
|> >
|> > > -----Original Message-----
|> > > From: Scott M Stark [mailto:[EMAIL PROTECTED]]
|> > > Sent: Wednesday, February 21, 2001 12:00 PM
|> > > To: Castro, David
|> > > Subject: Re: [jBoss-Dev] CVS update: newsite/business
|> > > jboss-tomcat.html
|> > >
|> > >
|> > > It looks like to me that the Jdk12Interceptor is doing the
|> > > same thing. I'm running
|> > > the jps and tomcat-ear tests successfully with the
|> > > Jdk12Interceptor in place of
|> > > the ContextClassLoaderInterceptor. If you have a test case
|> > > that needs this
|> > > let me know.
|> > >
|> > > Also all of the patches you gave me are in cvs. It would be
|> > > good if you can
|> > > test the jaws metadata issue to ensure that nothing has been lost.
|> > >
|> > >
|> > > ----- Original Message -----
|> > > From: "Castro, David" <[EMAIL PROTECTED]>
|> > > To: "'JBoss-Dev'" <[EMAIL PROTECTED]>
|> > > Cc: <[EMAIL PROTECTED]>
|> > > Sent: Wednesday, February 21, 2001 6:59 AM
|> > > Subject: RE: [jBoss-Dev] CVS update: newsite/business
|> > > jboss-tomcat.html
|> > >
|> > >
|> > > > Also need to add
|> > > >
|> > > > org.jboss.tomcat.ContextClassLoaderInterceptor
|> > > >
|> > > > to the ContextInterceptors in server.xml.
|> > > >
|> > > > Please add that to the documentation.  Without it, the
|> classloader
|> > > > integration doesn't work.  (at least, it didn't work
|> > > without it last time I
|> > > > tried)
|> > > >
|> > > >
|> > >
|> > >
|> >
|>
|
|
|
|
|
|---------------------------------------------------------------------
|This message (including any attachments) contains confidential, proprietary
|or privileged information intended for a specific purpose and
|individual(s),
|and is protected by law.  If you receive this message in error, please
|immediately delete it and all copies of it from your system, destroy any
|hard copies of it and notify the sender.  Any unauthorized disclosure,
|copying or distribution of any part of this message, or the taking of any
|unauthorized action based on it, is strictly prohibited.
|


Reply via email to