See the bug for stack traces. This causes a very intermittent test failure:
when we call for `LOGGER.info` from our custom security manager, it goes into
formatter, which loads locale data, which causes class load, which goes back to
this custom security manager, starting the loop again.
I thi
On Tue, 12 Sep 2023 12:23:40 GMT, Jaikiran Pai wrote:
>> See the bug for stack traces. This causes a very intermittent test failure:
>> when we call for `LOGGER.info` from our custom security manager, it goes
>> into formatter, which loads locale data, which causes class load, which goes
>> ba
e loop again.
>
> I think we get lucky most of the time to load the relevant classes outside
> this path. But sometimes, we don't get as lucky.
>
> Additional testing:
> - [x] 500x iterations of
> [JDK-8315640](https://bugs.openjdk.org/browse/JDK-8315640) reproducer
e loop again.
>
> I think we get lucky most of the time to load the relevant classes outside
> this path. But sometimes, we don't get as lucky.
>
> Additional testing:
> - [x] 500x iterations of
> [JDK-8315640](https://bugs.openjdk.org/browse/JDK-8315640) reproducer
On Tue, 12 Sep 2023 12:56:46 GMT, Aleksei Efimov wrote:
> Right, the `Logger` usages are not actually needed in this test - they can be
> replaced with `System.err::println`.
OK, I removed the logger from this test. But I would like to keep the common
branch free of any code to avoid any furth
On Tue, 12 Sep 2023 13:22:20 GMT, Aleksey Shipilev wrote:
>> See the bug for stack traces. This causes a very intermittent test failure:
>> when we call for `LOGGER.info` from our custom security manager, it goes
>> into formatter, which loads locale data, which causes clas
On Tue, 12 Sep 2023 16:58:59 GMT, Aleksey Shipilev wrote:
>> Aleksey Shipilev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Replace logger with System.err
>
> Thanks! I think I can integrate now, or do we
On Tue, 12 Sep 2023 11:32:46 GMT, Aleksey Shipilev wrote:
> See the bug for stack traces. This causes a very intermittent test failure:
> when we call for `LOGGER.info` from our custom security manager, it goes into
> formatter, which loads locale data, which causes class load, which
On Wed, 18 Sep 2024 10:31:59 GMT, serhiysachkov wrote:
> 8338759: Add extra diagnostic to java/net/InetAddress/ptr/Lookup.java
Marked as reviewed by shade (Reviewer).
-
PR Review: https://git.openjdk.org/jdk/pull/21055#pullrequestreview-2314827824
