On Thu, 10 Aug 2023 11:28:39 GMT, Raffaello Giulietti <rgiulie...@openjdk.org> 
wrote:

>>> AFAIK, if you need reproducible randoms in tests, you should add the tags:
>>> 
>>> ```
>>>  * @key randomness
>>>  * @library /test/lib
>>> ```
>>> 
>>> and initialize your random generator with
>>> 
>>> ```
>>> import jdk.test.lib.RandomFactory;
>>> ...
>>> Random rnd = RandomFactory.getRandom();
>>> ```
>>> 
>>> This prints the seed to STDOUT.
>> 
>> It'd be applicable for tests, but not for benchmarks. Also, the point that 
>> @cl4es seems to make is that randomness is captured and set up to reduce 
>> noise, not to be able to reproduce "benchmark failures".
>
> My bad. Forget my note

> Some have a preference for providing a seed for `Random` instances in micros. 
> Either hard-coded or through a `@Param` (I find this a bit excessive). Doing 
> so might reduce run-to-run noise.

I hear you, but unless your opinion is strong, I'll leave it as is.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14630#discussion_r1289994437

Reply via email to