Message10713
Looking at issue 2280 in detail again I agree that our switch to a non-blocking lock in Closer.java should solve 2280 too, given that it also involved Closer.dismiss and Closer.call.
So in principle I'd be willing to accept your (@Nick) patch. I'm a bit concerned though that we don't have handy unittests for this stuff.
So before accepting it I'd like to
- have Jim review it since he applied the original fix of issue 2280
- assess whether there's a difference to "just" reverting the patch for 2280 (which now appears to be obsolete) and - if yes - what, why etc
- understand/solve why the pullrequest triggers this error message "Your tests failed on CircleCI"
Nick, I suppose you're the best person to help answer these quickly, then we're ready to go. |
|
Date |
User |
Action |
Args |
2016-02-03 22:24:58 | stefan.richthofer | set | messageid: <1454538298.76.0.524272522709.issue2457@psf.upfronthosting.co.za> |
2016-02-03 22:24:58 | stefan.richthofer | set | recipients:
+ stefan.richthofer, zyasoft, nickmbailey |
2016-02-03 22:24:58 | stefan.richthofer | link | issue2457 messages |
2016-02-03 22:24:58 | stefan.richthofer | create | |
|