Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Date: Sun, 29 Oct 2017 03:54:40 +0000 (UTC)
From: "Bruno P. Kinoshita" <brunodepaulak@yahoo.com.br.INVALID>
Reply-To: "Bruno P. Kinoshita" <brunodepaulak@yahoo.com.br>
To: Commons Developers List <dev@commons.apache.org>
Message-ID: <1372189625.6903078.1509249280575@mail.yahoo.com>
In-Reply-To: <CACZkXPw=5zJ=Ox-Q-XCoKmax17h49QJay3uZ25SWX-m3zVJQ8w@mail.gmail.com>
References: <CACZkXPyVJ+MsTpEfaRspZE-bCC5viHD4FfLKTs268hCZYxah_g@mail.gmail.com> <CACZkXPwasziGvDxot3DNiknKomD65ukyk8v+gZGqA+mMRwyK9g@mail.gmail.com> <CACZkXPxE1wV62AQRv=je9o70jGtjET25Q3jfWftbPNX6XfYvLQ@mail.gmail.com> <CACZkXPyRYejUwvZM+svmOtVgLWftOR82HsF1xUCUogKn7b34dA@mail.gmail.com> <CACZkXPziHKq-u0gqQKr5eO958k=NFozuKXQG4Z3mkaV91sJm=w@mail.gmail.com> <CACZkXPw=5zJ=Ox-Q-XCoKmax17h49QJay3uZ25SWX-m3zVJQ8w@mail.gmail.com>
Subject: Re: [dbcp] update to pool 2.4.3.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
archived-at: Sun, 29 Oct 2017 03:58:58 -0000

Hi Gary,

Started the tests in Maven command line, found which tests failed. Executed the tests in Eclipse, found which class was related to the failure. Then did a diff between both tags.

    git diff POOL_2_4_2 POOL_2.4.3-RC1 -- ./src/main/java/org/apache/commons/pool2/impl/DefaultPooledObject.java

The short diff indicates that the failures started possibly due to the replacement of Exceptions in the DefaultPooledObject by a CallStack. Looks like the CallStack does not keep track of all the methods called (due to a security context manager from what I could tell?). Only classes.

The following pull request updates pool to 2.4.3, and changes the unit tests to, instead of looking for method names in the log writer output, to look for the test class name.

    https://github.com/apache/commons-dbcp/pull/8


Feel free to review and merge if you agree it's a good solution.
Hope that helps,
Bruno

ps: I think we need to publish the commons-pool website. Tried to have a look at the changes report for 2.4.3 but it wasn't there yet.
ps2: I'm currently looking at those trivial checkstyle issues that we found during the vote thread. Going to send another pull request for that today


________________________________
From: Gary Gregory <garydgregory@gmail.com>
To: Commons Developers List <dev@commons.apache.org> 
Sent: Sunday, 29 October 2017 8:44 AM
Subject: [dbcp] update to pool 2.4.3.


Hi All,


After updating dbcp to 2.4.3 I am seeing two unit test failures. I have to

step out for the day and I am hoping someone can help figure out what is

going on.


Gary

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org