You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by bu...@apache.org on 2005/09/20 08:07:11 UTC

DO NOT REPLY [Bug 36719] - Most ObjectPool.borrowObject() implementations do not call validateObject()

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=36719>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=36719





------- Additional Comments From Sandy@McArthur.org  2005-09-20 08:07 -------
Created an attachment (id=16453)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=16453&action=view)
borrowObject() fixes and unit tests

With this patch
TestStackObjectPool.testBorrowReturnWithSometimesInvalidObjects() fails. I
think this test is broken or at least it shouldn't have the
assertEquals(3,pool.getNumIdle()); at the end. Best case this test was intended
to check for unexpected exceptions and the assertEquals isn't needed or worst
case the test incorrectly asserts that broken behavior is valid. Please let me
know if I misunderstood it's intent.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

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