You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bu...@apache.org on 2014/04/11 22:03:18 UTC

[Bug 56400] New: Change POOL2/DBCP2 consumption strategy

https://issues.apache.org/bugzilla/show_bug.cgi?id=56400

            Bug ID: 56400
           Summary: Change POOL2/DBCP2 consumption strategy
           Product: Tomcat 8
           Version: trunk
          Hardware: All
                OS: All
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Meta
          Assignee: dev@tomcat.apache.org
          Reporter: chris@christopherschultz.net

Use package-renamed version as with other commons components.

-- 
You are receiving this mail because:
You are the assignee for the bug.

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


[Bug 56400] Change POOL2/DBCP2 consumption strategy

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=56400

Mark Thomas <ma...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #3 from Mark Thomas <ma...@apache.org> ---
1 through 3 fixed.

4. I don't think so.
If we start making functional changes (why would we?) then yes, but otherwise I
don't see a need to make our unit tests run for even longer than they do
already when Commons is already testing this code.

-- 
You are receiving this mail because:
You are the assignee for the bug.

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


[Bug 56400] Change POOL2/DBCP2 consumption strategy

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=56400

Konstantin Kolinko <kn...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---

--- Comment #2 from Konstantin Kolinko <kn...@gmail.com> ---
1. Fore a record,
 'ant release' with Java 8 completes successfully.

Good.
So we are not limited to Java 7 as it were with DBCP 1.

2. There are several minor cleanups pending:
a. Correct DBCP description in

BUILDING.txt

b. Remove DBCP and pool versions from

build.properties.default

c. Remove TOMCAT_LIBS_BASE/tomcat8-deps/dbcp/tomcat-dbcp.jar reference from

res/ide-support/eclipse/eclipse.classpath

The IDEA and NetBeans files look OK.


3. There is javadoc link to an absent class (ProxiedObjectPool) in
dbcp2/package-info.java (see buildbot output for 'ant release' step)


4. I wonder whether it is worth and feasible to fork the tests as well.

-- 
You are receiving this mail because:
You are the assignee for the bug.

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