Hi Mandy,
I've incorporated your changes, run JPRT, and posted an updated webrev:
http://cr.openjdk.java.net/~jgish/Bug8010939-LogManager-Deadlock.1/
http://cr.openjdk.java.net/%7Ejgish/Bug8010939-LogManager-Deadlock.1/
Thanks,
Jim
On 04/12/2013 06:51 PM, Mandy Chung wrote:
Hi Jim,
Please review:
http://cr.openjdk.java.net/~jgish/Bug8010939-LogManager-Deadlock/
http://cr.openjdk.java.net/%7Ejgish/Bug8010939-LogManager-Deadlock/
Thanks,
Jim
--
Jim Gish | Consulting Member of Technical Staff | +1.781.442.0304
Oracle Java Platform Group | Core Libraries Team
35 Network
Hi Jim,
Thanks for fixing this.
On 4/12/2013 1:11 PM, Jim Gish wrote:
Please review:
http://cr.openjdk.java.net/~jgish/Bug8010939-LogManager-Deadlock/
The fix looks okay. I would recommend to move the call to
manager.drainLoggerRefQueueBounded() back to LogManager.addLogger which
was