On Fri, 5 Jun 2026 12:43:44 GMT, Jaikiran Pai <[email protected]> wrote:
>> test/jdk/java/util/zip/GZIP/GZIPOverBlockingStreams.java line 69:
>>
>>> 67: private static final Random random = RandomFactory.getRandom();
>>> 68: private static final String MEMBER_CONTENT_FORMAT =
>>> 69: "Hello member %d, greetings from Java " +
>>> System.getProperty("java.version") + "\n";
>>
>> Out of curiosity, could including `System.getProperty("java.version")` risk
>> that the test becomes unstable between Java versions?
>>
>> Would it be better to obtain any 'random' data from the already existing
>> `random`?
>
> Hello @Marcono1234, the use of `java.version` value in that text is just for
> arbitrary text. That value doesn't get compared with any Java version. In any
> case, to avoid confusion, I'll remove that part from the text message.
Done, I've updated the PR to replace with a differen text.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/31369#discussion_r3362725910
