RFR: 8060132: Handlers configured on abstract nodes in logging.properties are not always properly closed

Daniel Fuchs daniel.fuchs at oracle.com
Fri Oct 10 15:39:55 UTC 2014


On 10/10/14 17:10, Stanimir Simeonoff wrote:
> persistentLoggers should be cleared on reset(), imo.

Yes - I was wondering about that too, it's a bit tricky to
get it right (WRT MT-safety) :-(.

> Also using "count" and then "for in" to loop over an array looks a bit
> ugly, should be just "for (int i=0; i<names.length; i++){ final String
> className = names[i];..}

I wanted to add the logger only if at least one handler was
successfully created and added, so 'count' is not necessarily
equals to 'i'.

> Calling the class name 'word' is weird as well. (I do understand that's
> not new but it's a good time to get it straight).

Agreed.

http://cr.openjdk.java.net/~dfuchs/webrev_8060132/webrev.01

best regards, and thanks for all the valuable inputs & comment from
you and Jason - I really appreciate it :-)

-- daniel

>
> Cheers
> Stanimir
>
>
> On Fri, Oct 10, 2014 at 5:51 PM, Daniel Fuchs <daniel.fuchs at oracle.com
> <mailto:daniel.fuchs at oracle.com>> wrote:
>
>     Hi,
>
>     Please find below a possible fix for:
>
>     8060132: Handlers configured on abstract nodes in logging.properties
>     are not always properly closed
>     https://bugs.openjdk.java.net/__browse/JDK-8060132
>     <https://bugs.openjdk.java.net/browse/JDK-8060132>
>
>     webrev:
>     http://cr.openjdk.java.net/~__dfuchs/webrev_8060132/webrev.__00/
>     <http://cr.openjdk.java.net/~dfuchs/webrev_8060132/webrev.00/>
>
>     Other options have been discussed in this other email thread:
>
>     Subject: JDK-6774110 lock file is not deleted when child logger is used
>     http://mail.openjdk.java.net/__pipermail/core-libs-dev/2014-__October/029038.html
>     <http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-October/029038.html>
>
>     best regards,
>
>     -- daniel
>
>




More information about the core-libs-dev mailing list