RFR: TEST_BUG: 8222027 - java/util/logging/LogManager/TestLoggerNames.java generates intermittent ClassCastException

Steve Groeger GROEGES at uk.ibm.com
Fri Apr 5 13:01:41 UTC 2019


Hi Daniel, 

Thanks. I should have spotted that slight mistake. That is what comes from 
just cutting and pasting.
Have created a new webrev anyway: 
http://cr.openjdk.java.net/~sgroeger/8222027/webrev.02/
I presume I need someone else to review this and also get a sponsor (so 
they can actually make the change) as I am not a committer (YET!!!)
Do you know anyone that can do this for me?

Thanks
Steve Groeger
IBM Runtime Technologies
Hursley, Winchester
Tel: (44) 1962 816911  Mobex: 279990  Mobile: 07718 517 129
Fax (44) 1962 816800
Lotus Notes: Steve Groeger/UK/IBM
Internet: groeges at uk.ibm.com

Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 
741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU



From:   Daniel Fuchs <daniel.fuchs at oracle.com>
To:     Steve Groeger <GROEGES at uk.ibm.com>
Cc:     OpenJDK Core Libs Developers <core-libs-dev at openjdk.java.net>
Date:   05/04/2019 13:51
Subject:        Re: RFR: TEST_BUG: 8222027 - 
java/util/logging/LogManager/TestLoggerNames.java generates intermittent 
ClassCastException



Hi Steve,

On 05/04/2019 13:14, Steve Groeger wrote:
> Hi Daniel,
> 
> Thanks for the quick review.
> I too didnt see why the test wanted to obtain the TestLogger from the 
> LogManager, but I left it doing that just in case there was a reason.
> If you think this is not needed then I agree with your suggested 
> modifications. I have created a new webrev here:
> 
_https://urldefense.proofpoint.com/v2/url?u=http-3A__cr.openjdk.java.net_-7Esgroeger_8222027_webrev.01_-5F&d=DwID-g&c=jf_iaSHvJObTbx-siA1ZOg&r=78GW2OHz7nNTH2dBkTx7-TKh2QCt3JD3zukzeUO8RpA&m=Df0w0-wNA_rWl-TkWAFLlG6aQu1TT5WHca1Uzlnx8G0&s=Fcqc74kaTxOKtJS9P_kQ-qQak7U377oypPD77yt38gM&e=

Perfect! Maybe remove the trailing " for" in
  throw new AssertionError("wrong logger returned for");
Sorry - that was my mistake...

No need for a new webrev.

Reviewed.

best regards,

-- daniel

> Hopefully I understood what you were saying and these modification are 
> OK. Please can you re-review these and let me know if OK.
> 
> Thanks
> Steve Groeger
> IBM Runtime Technologies
> Hursley, Winchester
> Tel: (44) 1962 816911  Mobex: 279990  Mobile: 07718 517 129
> Fax (44) 1962 816800
> Lotus Notes: Steve Groeger/UK/IBM
> Internet: groeges at uk.ibm.com
> 
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number 

> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 
3AU
> 
> 
> 
> From: Daniel Fuchs <daniel.fuchs at oracle.com>
> To: Steve Groeger <GROEGES at uk.ibm.com>, OpenJDK Core Libs Developers 
> <core-libs-dev at openjdk.java.net>
> Date: 05/04/2019 12:33
> Subject: Re: RFR: TEST_BUG: 8222027 - 
> java/util/logging/LogManager/TestLoggerNames.java generates intermittent 

> ClassCastException
> ------------------------------------------------------------------------
> 
> 
> 
> Hi Steve,
> 
> Good analysis! I believe we can make the fix even
> more simpler by simply moving:
> 
> TestLogger test = new TestLogger(..);
> LogManager.getLogManager().addLogger(test);
> 
> at the beginning of main. If we do that then we can get
> rid of the new local variable 'tl' and of the
> reachabilityFence() call.
> 
> I see no real reason why the test wants to obtain
> the TestLogger test from the LogManager.
> 
> If you wanted to be extra cautious in preserving the original
> test then you could also add:
> 
> if (test != Logger.getLogger("com.foo.bar.zzz")) {
>      throw new AssertionError("wrong logger returned for");
> }
> 
> in place of line 149...
> 
> best regards,
> 
> -- daniel
> 
> On 05/04/2019 11:59, Steve Groeger wrote:
>  > Hi all,
>  >
>  > Please could I have a review for this bug:
>  > 
https://urldefense.proofpoint.com/v2/url?u=https-3A__bugs.openjdk.java.net_browse_JDK-2D8222027&d=DwID-g&c=jf_iaSHvJObTbx-siA1ZOg&r=78GW2OHz7nNTH2dBkTx7-TKh2QCt3JD3zukzeUO8RpA&m=Df0w0-wNA_rWl-TkWAFLlG6aQu1TT5WHca1Uzlnx8G0&s=ZA3MXC7DrYMf7UBacMB-BeS9MVxWoZvSIrcuzPjhW8o&e=

>  >
>  > The webrev is here:
>  > 
https://urldefense.proofpoint.com/v2/url?u=http-3A__cr.openjdk.java.net_-7Esgroeger_8222027_webrev.00_&d=DwID-g&c=jf_iaSHvJObTbx-siA1ZOg&r=78GW2OHz7nNTH2dBkTx7-TKh2QCt3JD3zukzeUO8RpA&m=Df0w0-wNA_rWl-TkWAFLlG6aQu1TT5WHca1Uzlnx8G0&s=aQ7Xz4DUgXgJDrV6Y-ZE0AiDjJtOugZv1StE-2kuff4&e=

>  >
>  > Thanks
>  > Steve Groeger
>  > IBM Runtime Technologies
>  > Hursley, Winchester
>  > Tel: (44) 1962 816911  Mobex: 279990  Mobile: 07718 517 129
>  > Fax (44) 1962 816800
>  > Lotus Notes: Steve Groeger/UK/IBM
>  > Internet: groeges at uk.ibm.com
>  >
>  > Unless stated otherwise above:
>  > IBM United Kingdom Limited - Registered in England and Wales with 
number
>  > 741598.
>  > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire 
> PO6 3AU
>  > Unless stated otherwise above:
>  > IBM United Kingdom Limited - Registered in England and Wales with 
number
>  > 741598.
>  > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire 
> PO6 3AU
>  >
> 
> 
> 
> 
> 
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with number 

> 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 
3AU





Unless stated otherwise above:
IBM United Kingdom Limited - Registered in England and Wales with number 
741598. 
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU


More information about the core-libs-dev mailing list