You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Florent Guillaume <fg...@nuxeo.com> on 2020/12/09 00:18:56 UTC

[DBCP] ManagedConnection must clear its cached state after transaction completes

Hi,

A few weeks ago I opened the ticket
https://issues.apache.org/jira/browse/DBCP-568 (ManagedConnection must
clear its cached state after transaction completes)
with an associated proposed fix at
https://github.com/apache/commons-dbcp/pull/75.
Would anyone be interested in reviewing this?

Thanks,
Florent

-- 
[image: Nuxeo Logo] <https://www.nuxeo.com/>

Florent Guillaume  Head of R&D  [image: LinkedIn]
<https://www.linkedin.com/in/fguillaume/> [image: Twitter]
<https://twitter.com/efge> [image: Github] <https://github.com/efge>

Nuxeo Content Services Platform. Stay ahead.

Re: [DBCP] ManagedConnection must clear its cached state after transaction completes

Posted by Florent Guillaume <fg...@nuxeo.com>.
Hi,

You're right. I've now added a test in the PR.

Thanks,
Florent


On Thu, Dec 10, 2020 at 8:55 PM Gary Gregory <ga...@gmail.com> wrote:

> Hi Florent,
>
> There has to be a way to test SOMETHING about this, using H2 for example,
> or mocking, otherwise, this is just a regression bug waiting to happen.
>
> Gary
>
> On Tue, Dec 8, 2020 at 7:19 PM Florent Guillaume <fg...@nuxeo.com>
> wrote:
>
> > Hi,
> >
> > A few weeks ago I opened the ticket
> > https://issues.apache.org/jira/browse/DBCP-568 (ManagedConnection must
> > clear its cached state after transaction completes)
> > with an associated proposed fix at
> > https://github.com/apache/commons-dbcp/pull/75.
> > Would anyone be interested in reviewing this?
> >
> > Thanks,
> > Florent
> >
> > --
> > [image: Nuxeo Logo] <https://www.nuxeo.com/>
> >
> > Florent Guillaume  Head of R&D  [image: LinkedIn]
> > <https://www.linkedin.com/in/fguillaume/> [image: Twitter]
> > <https://twitter.com/efge> [image: Github] <https://github.com/efge>
> >
> > Nuxeo Content Services Platform. Stay ahead.
> >
>


-- 
[image: Nuxeo Logo] <https://www.nuxeo.com/>

Florent Guillaume  Head of R&D  [image: LinkedIn]
<https://www.linkedin.com/in/fguillaume/> [image: Twitter]
<https://twitter.com/efge> [image: Github] <https://github.com/efge>

Nuxeo Content Services Platform. Stay ahead.

Re: [DBCP] ManagedConnection must clear its cached state after transaction completes

Posted by Gary Gregory <ga...@gmail.com>.
Hi Florent,

There has to be a way to test SOMETHING about this, using H2 for example,
or mocking, otherwise, this is just a regression bug waiting to happen.

Gary

On Tue, Dec 8, 2020 at 7:19 PM Florent Guillaume <fg...@nuxeo.com>
wrote:

> Hi,
>
> A few weeks ago I opened the ticket
> https://issues.apache.org/jira/browse/DBCP-568 (ManagedConnection must
> clear its cached state after transaction completes)
> with an associated proposed fix at
> https://github.com/apache/commons-dbcp/pull/75.
> Would anyone be interested in reviewing this?
>
> Thanks,
> Florent
>
> --
> [image: Nuxeo Logo] <https://www.nuxeo.com/>
>
> Florent Guillaume  Head of R&D  [image: LinkedIn]
> <https://www.linkedin.com/in/fguillaume/> [image: Twitter]
> <https://twitter.com/efge> [image: Github] <https://github.com/efge>
>
> Nuxeo Content Services Platform. Stay ahead.
>