From: Sacha L. <Sac...@ml...> - 2002-07-03 19:06:09
|
Ah, Ok, I hadn't seen that. So, I guess it stops the discussion on HEAD. Thank you for these changes Dain, it is a lot more clear like that now! Cheers, Sacha > -----Message d'origine----- > De : jbo...@li... > [mailto:jbo...@li...]De la part de Dain > Sundstrom > Envoyé : mercredi, 3 juillet 2002 20:52 > À : jbo...@li... > Objet : Re: [JBoss-user] java.rmi.ServerException: null > > > I rewrote all of the exception handling earlier this week. The new code > follows: > > // We inherited tx: Tell caller we marked for rollback only. > if (isLocal) > { > throw new TransactionRolledbackLocalException( > cause.getMessage(), > (Exception)cause); > } > else > { > TransactionRolledbackException ex = > new TransactionRolledbackException(cause.getMessage()); > ex.detail = cause; > throw ex; > } > > I don't remember what it looked like before, but most of the exception > handling was terrible. I only made changes to HEAD as there were a lot > of files changed, and back porting is not easy anymore. > > -dain > > Sacha Labourey wrote: > > Yes Dain, > > > > But if you take a look at the code that is involved, you will > see that in > > the Tx interceptor, the exception is wrapped in a > ServerException and the > > source exception is simply forgotten: > > > > throw ex; > > } > > // OSH: Should this be wrapped? > > ! ex = new ServerException(e.getMessage()); > > } else > > // We inherited tx: Tell caller we marked for rollback > > only. > > ex = new > TransactionRolledbackException(e.getMessage()); > > > > > > the code takes the e.getMessage from the source exception as > the message of > > the new exception (called "ex") but that's all. I agree with Marius: > > something needs to be done. With Jdk1.4, the best way would be > to set the > > cause exception of "ex" to be "e" but as this is non-portable > with 1.3.x, we > > should do more than simply display the message. > > > > Cheers, > > > > > > Sacha > > > > > > > >>-----Message d'origine----- > >>De : jbo...@li... > >>[mailto:jbo...@li...]De la part de Dain > >>Sundstrom > >>Envoyé : mercredi, 3 juillet 2002 20:14 > >>À : jbo...@li... > >>Objet : Re: [JBoss-user] java.rmi.ServerException: null > >> > >> > >>There is no magic. All system exceptions are automatically logged by > >>the LogInterceptor. Application exceptions are only logged if the call > >>tracing is turned on for the LogInterceptor, but this is not an > >>application exception. > >> > >>-dain > >> > >>JD Brennan wrote: > >> > >>>So what's the magic change to log4j.xml that I set > >>>to get a stack trace in this case? I've seen this too. > >>> > >>>Tx! > >>>JD > >>> > >>>-----Original Message----- > >>>From: ma...@bo... [mailto:ma...@bo...] > >>>Sent: Wednesday, July 03, 2002 9:58 AM > >>>To: jbo...@li... > >>>Subject: Re: [JBoss-user] java.rmi.ServerException: null > >>> > >>> > >>> > >> > >>http://sourceforge.net/tracker/index.php?func=detail&aid=562036&gr > >>oup_id=22866&atid=376687 > >> > >><http://sourceforge.net/tracker/index.php?func=detail&aid=562036&g > >>roup_id=22866&atid=376687> > >> > >>> > >>>On Wed, Jul 03, 2002 at 04:20:34PM +0200, Sacha Labourey wrote: > >>> > Which patch? > >>> > > >>> > > -----Message d'origine----- > >>> > > De : jbo...@li... > >>> > > [mailto:jbo...@li...]De la part de > >>> > > ma...@bo... > >>> > > Envoyé : mercredi, 3 juillet 2002 16:14 > >>> > > À : jbo...@li... > >>> > > Objet : Re: [JBoss-user] java.rmi.ServerException: null > >>> > > > >>> > > > >>> > > Not to me either. Therefore I made a patch, but Scott Stark > >>> > > refused to apply it. He said that it was passible to get a better > >>> > > stacktrace by changing log4j.xml, but would unfortunately not set > >>> > > that as default :-( > >>> > > > >>> > > > >>> > > On Wed, Jul 03, 2002 at 03:02:09PM +0200, Michael Delamere wrote: > >>> > > > thanks, > >>> > > > > >>> > > > unforunately the error message doesn´t tell me very much :-) ! > >>> > > > > >>> > > > Regards, > >>> > > > > >>> > > > Michael > >>> > > > > >>> > > > > >>> > > > ----- Original Message ----- > >>> > > > From: <ma...@bo...> > >>> > > > To: <jbo...@li...> > >>> > > > Sent: Wednesday, July 03, 2002 2:21 PM > >>> > > > Subject: Re: [JBoss-user] java.rmi.ServerException: null > >>> > > > > >>> > > > > >>> > > > > It is a runtime exception (like nullpointer, > >> > >>indexoutofbounds etc) > >> > >>> > > > happening in your bean. > >>> > > > > > >>> > > > > On Wed, Jul 03, 2002 at 01:48:13PM +0200, Michael > >> > >>Delamere wrote: > >> > >>> > > > > > Hi, > >>> > > > > > > >>> > > > > > When accessing my entity bean via my stateless session bean > >>> > > I get the > >>> > > > > > following error: > >>> > > > > > > >>> > > > > > ============================================= > >>> > > > > > 13:50:23,113 INFO [STDOUT] java.rmi.ServerException: null > >>> > > > > > Embedded Exception > >>> > > > > > null; nested exception is: > >>> > > > > > javax.ejb.EJBException: null > >>> > > > > > Embedded Exception > >>> > > > > > null > >>> > > > > > > >>> > > > > > ============================================= > >>> > > > > > > >>> > > > > > Is this a common error that others have experienced or will > >>> > > I have to > >>> > > > dig > >>> > > > > > deeper? > >>> > > > > > > >>> > > > > > Thanks, > >>> > > > > > > >>> > > > > > Michael > >>> > > > > > > >>> > > > > > > >>> > > > > > > >>> > > > > > ------------------------------------------------------- > >>> > > > > > This sf.net email is sponsored by:ThinkGeek > >>> > > > > > No, I will not fix your computer. > >>> > > > > > http://thinkgeek.com/sf > >>> > > > > > _______________________________________________ > >>> > > > > > JBoss-user mailing list > >>> > > > > > JBo...@li... > >>> > > > > > https://lists.sourceforge.net/lists/listinfo/jboss-user > >>> > > > > > >>> > > > > -- > >>> > > > > MVH > >>> > > > > Marius Kotsbak > >>> > > > > Boost communications AS > >>> > > > > > >>> > > > > > >>> > > > > ------------------------------------------------------- > >>> > > > > This sf.net email is sponsored by:ThinkGeek > >>> > > > > No, I will not fix your computer. > >>> > > > > http://thinkgeek.com/sf > >>> > > > > _______________________________________________ > >>> > > > > JBoss-user mailing list > >>> > > > > JBo...@li... > >>> > > > > https://lists.sourceforge.net/lists/listinfo/jboss-user > >>> > > > > >>> > > > > >>> > > > > >>> > > > ------------------------------------------------------- > >>> > > > This sf.net email is sponsored by:ThinkGeek > >>> > > > No, I will not fix your computer. > >>> > > > http://thinkgeek.com/sf > >>> > > > _______________________________________________ > >>> > > > JBoss-user mailing list > >>> > > > JBo...@li... > >>> > > > https://lists.sourceforge.net/lists/listinfo/jboss-user > >>> > > > >>> > > -- > >>> > > MVH > >>> > > Marius Kotsbak > >>> > > Boost communications AS > >>> > > > >>> > > > >>> > > ------------------------------------------------------- > >>> > > This sf.net email is sponsored by:ThinkGeek > >>> > > No, I will not fix your computer. > >>> > > http://thinkgeek.com/sf > >>> > > _______________________________________________ > >>> > > JBoss-user mailing list > >>> > > JBo...@li... > >>> > > https://lists.sourceforge.net/lists/listinfo/jboss-user > >>> > > > >>> > > >>> > > >>> > > >>> > ------------------------------------------------------- > >>> > This sf.net email is sponsored by:ThinkGeek > >>> > No, I will not fix your computer. > >>> > http://thinkgeek.com/sf > >>> > _______________________________________________ > >>> > JBoss-user mailing list > >>> > JBo...@li... > >>> > https://lists.sourceforge.net/lists/listinfo/jboss-user > >>> > >>>-- > >>>MVH > >>>Marius Kotsbak > >>>Boost communications AS > >>> > >>> > >>>------------------------------------------------------- > >>>This sf.net email is sponsored by:ThinkGeek > >>>No, I will not fix your computer. > >>>http://thinkgeek.com/sf > >>>_______________________________________________ > >>>JBoss-user mailing list > >>>JBo...@li... > >>>https://lists.sourceforge.net/lists/listinfo/jboss-user > >>> > >> > >> > >>-- > >>xxxxxxxxxxxxxxxxxxxxxxxx > >>Dain Sundstrom > >>Chief Architect JBossCMP > >>JBoss Group, LLC > >>xxxxxxxxxxxxxxxxxxxxxxxx > >> > >> > >> > >>------------------------------------------------------- > >>This sf.net email is sponsored by:ThinkGeek > >>No, I will not fix your computer. > >>http://thinkgeek.com/sf > >>_______________________________________________ > >>JBoss-user mailing list > >>JBo...@li... > >>https://lists.sourceforge.net/lists/listinfo/jboss-user > >> > > > > > > > > > > ------------------------------------------------------- > > This sf.net email is sponsored by:ThinkGeek > > No, I will not fix your computer. > > http://thinkgeek.com/sf > > _______________________________________________ > > JBoss-user mailing list > > JBo...@li... > > https://lists.sourceforge.net/lists/listinfo/jboss-user > > > -- > xxxxxxxxxxxxxxxxxxxxxxxx > Dain Sundstrom > Chief Architect JBossCMP > JBoss Group, LLC > xxxxxxxxxxxxxxxxxxxxxxxx > > > > ------------------------------------------------------- > This sf.net email is sponsored by:ThinkGeek > No, I will not fix your computer. > http://thinkgeek.com/sf > _______________________________________________ > JBoss-user mailing list > JBo...@li... > https://lists.sourceforge.net/lists/listinfo/jboss-user > |