RFR: 8048020 - Regression on java.util.logging.FileHandler
jason_mehrens at hotmail.com
Tue Jul 1 13:40:32 UTC 2014
Looks bullet proof to me.
> Date: Tue, 1 Jul 2014 11:25:33 +0200
> From: daniel.fuchs at oracle.com
> To: jason_mehrens at hotmail.com; core-libs-dev at openjdk.java.net
> CC: alan.bateman at oracle.com
> Subject: Re: RFR: 8048020 - Regression on java.util.logging.FileHandler
> On 6/25/14 2:34 PM, Jason Mehrens wrote:
>> FileChannel.open(WRITE,APPEND) could throw NoSuchFileException during a startup and shutdown race between two VMs. That case needs to either perform a bounded retry or continue and rotate.
> Hi Jason, Alan,
> Here is an updated version of the fix that does a bounded
> retry (retries once - and if it fails, proceeds with the next
> name). I have also added NO_FOLLOW_LINKS - for the case where
> we try to open an existing Lockfile, and suppressed the
> Files.isWritable check since that will be taken care of by
> the call to FileChannel.open.
> I also updated the comments to make it clear that the
> file could not have been locked by another instance
> of FileHandler (since that would have been taken care
> of by our global 'locks' synchronization).
> best regards,
> -- daniel
>>> Date: Tue, 24 Jun 2014 20:34:46 +0200
>>> From: daniel.fuchs at oracle.com
>>> To: core-libs-dev at openjdk.java.net
>>> CC: jason_mehrens at hotmail.com; Alan.Bateman at oracle.com
>>> Subject: RFR: 8048020 - Regression on java.util.logging.FileHandler
>>> Please find below a patch  for :
>>> 8048020 - Regression on java.util.logging.FileHandler
>>> This corresponds to the problem that was discussed on this list
>>> in the following email thread:
>>> "Zombie FileHandler locks can exhaust all available log file locks."
>>>  http://cr.openjdk.java.net/~dfuchs/webrev_8048020/webrev.00/
>>>  https://bugs.openjdk.java.net/browse/JDK-8048020
>>> This patch would be a good candidate for being backported on 8, since
>>> the issue was introduced in 8.
>>> best regards,
>>> -- daniel
More information about the core-libs-dev