garydgregory commented on PR #230:
URL: https://github.com/apache/commons-pool/pull/230#issuecomment-1635792119

   @zenghu1chen 
   Thank you for the PR.
   Please see the build failure and update the PR.
   There are no unit tests here, I'd really like to see a test that can 
demonstrate a dead lock in order to prove that the changes actually fix an 
issue rather than relying on some external empirical evidence. The test need 
not use Redis for example but be built in whatever way effectively causes a 
test to fail or hang.
   CC: @psteitz 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to