You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@commons.apache.org by "garydgregory (via GitHub)" <gi...@apache.org> on 2023/07/14 12:27:14 UTC

[GitHub] [commons-pool] garydgregory commented on pull request #230: Support for loom

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: issues-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org