Mailing-List: contact users-help@tomcat.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Tomcat Users List" <users@tomcat.apache.org>
Received-SPF: pass (nike.apache.org: domain of amits.84@gmail.com designates
 209.85.220.173 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <4F577D83.6040601@christopherschultz.net>
References: 
 <CALpK9-SjWD3Eq4TStgsDKQi-8xcFZCHn3ADq-RmSBcjyg05Urw@mail.gmail.com>
 <4F56212E.6010208@christopherschultz.net>
 <CALpK9-SZ4vFXp4dqdHOmLkW5nGh3qDoi9F-yqG9ivBW=sh+R0Q@mail.gmail.com>
 <4F577D83.6040601@christopherschultz.net>
From: amit shah <amits.84@gmail.com>
Date: Thu, 8 Mar 2012 14:04:06 +0530
Message-ID: 
 <CALpK9-St4VPrwRh0Dhp-osB5T8DccAEYde6_MPkR29YmeHRc2Q@mail.gmail.com>
Subject: Re: JDBC Pool - Error handling during connection creation
To: Tomcat Users List <users@tomcat.apache.org>
Content-Type: multipart/alternative; boundary=20cf30780aced13e2004bab7248a

--20cf30780aced13e2004bab7248a
Content-Type: text/plain; charset=ISO-8859-1

Comment below

On Wed, Mar 7, 2012 at 8:53 PM, Christopher Schultz <
chris@christopherschultz.net> wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Amit,
>
> On 3/7/12 12:12 AM, amit shah wrote:
> > I am using tomcat-jdbc.jar and tomcat-juli.jar from version
> > 7.0.26.
> >
> >> I don't see any place in setupConnection where an exception is
> >> swallowed, do you?
> >>
> >>
> http://svn.apache.org/viewvc/tomcat/trunk/modules/jdbc-pool/src/main/java/org/apache/tomcat/jdbc/pool/ConnectionPool.java?view=markup
> >
> >>
> > Have a look at the public boolean validate(int
> > validateAction,String sql) method in PooledConnection class line no
> > - 445.
>
> > The validate method ignores any exception which is thrown while
> > validating the connection or executing the initSQL query.
>
> If you have debug logging enabled, it will log the exception. So,
> enable debug logging and try again. *shrug*
>
> > The createConnection() method in ConnectionPool class which calls
> > the validate() method returns null in such a case and hence it
> > leads to a null pointer exception in setupConnection method.
> >
> > Ignoring the exception in the validate() method may sound
> > appropriate (not sure for what reason though) but my point is that
> > it makes troubleshooting the issue a lot harder.
>
> Just turn on debug logging and you'll see the original exception. I
> think in this case, the error is quite fatal so checking for null and
> emitting some other error message would only be slightly more helpful
> to you, but the effect would be the same.
>

I understand that turning the log level to debug will display the original
exception. The point is in production environments the log level would be
by default info and it would take a longer time to understand the root
cause.

In our case, we execute a SP as an initSQL query, so the reasons why the
query fails would increase. Hence it would been easier if the validate()
method throws the exception instead of ignoring it.

Do you see any issues in throwing the exception right there?


>
> - -chris
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG/MacGPG2 v2.0.17 (Darwin)
> Comment: GPGTools - http://gpgtools.org
> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
>
> iEYEARECAAYFAk9XfYMACgkQ9CaO5/Lv0PBdqQCgjhbGeAQ2QU64XOrYP5VBEkF6
> VkoAoL6g5gMTaYpnHvj7HZDqWT6P+qcE
> =AzPZ
> -----END PGP SIGNATURE-----
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: users-unsubscribe@tomcat.apache.org
> For additional commands, e-mail: users-help@tomcat.apache.org
>
>

--20cf30780aced13e2004bab7248a--