RFR: 8253750: use build-stable default seed for Utils.RANDOM_GENERATOR [v3]

Roger Riggs rriggs at openjdk.java.net
Tue Oct 6 13:38:08 UTC 2020


On Mon, 5 Oct 2020 16:46:55 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:

>> Hi all,
>> 
>> could you please review the patch which updates `jdk.test.lib.Utils` to use md5 hash-sum of `java.vm.version` property
>> as default seed for `Utils.RANDOM_GENERATOR`?
>> from JBS:
>>> using the same seed for all runs of a build will make it possible (easier) to compare results from different test runs
>>> (e.g. on different platforms, w/ different flags) and consequently will make test results analysis easier. the proposed
>>> solution is to use the seed based on Runtime.version() / "java.vm.version", which are different from build to build, if
>>> there is no seed specified by "jdk.test.lib.random.seed" property.
>> 
>> the patch also updates `RandomGeneratorTest` test, so it expects now that the same values are generated if no seed is
>> provided.
>> testing:
>> ✅ tier1
>> ✅ `test/lib-test/jdk/test/lib/` against personal build on linux,windows,macos-x64
>> ✅ `test/lib-test/jdk/test/lib/` against CI build on linux,windows,macos-x64
>
> Igor Ignatyev has updated the pull request incrementally with one additional commit since the last revision:
> 
>   use random seed for personal/internal builds

Marked as reviewed by rriggs (Reviewer).

-------------

PR: https://git.openjdk.java.net/jdk/pull/391


More information about the core-libs-dev mailing list