dev@glassfish.java.net

Re: patch for issue #7522 - exceptions thrown from getInvalidConnections method are not logged

From: Shalini Muthukrishnan <Shalini.Muthukrishnan_at_Sun.COM>
Date: Wed, 26 Aug 2009 15:02:33 +0530

Hi Dies,

In your fix, please do not catch Exception, as the resource adapter as
per contract throws only ResourceException. So the first part is fine.

Thanks,
Shalini.

Dies Koper wrote:
> Hi Shalini,
>
> I finally finalized my patch (attached to the issue).
> I have split up the messages. In the case of ResourceException: I take
> advantage of RE's implementation of getMessage to include the error code.
>
> May I commit it to V3 trunk?
>
> Regards,
> Dies
>
>