RFR: CODETOOLS-7903748 - jcstress: Test list should honor concurrency settings [v6]

PM duke at openjdk.org
Thu Jul 4 08:21:35 UTC 2024


On Thu, 4 Jul 2024 08:11:44 GMT, Jiří Vaněk <jvanek at openjdk.org> wrote:

>> This is extracting  List<TestConfig> configs =prepareRunProgram(classes, tests);
>> with all he HW/switches setup to separated method and reusing it in `-l` mode
>> 
>> I'm aware of triplicated removal of -agentlib, and will clean it up as
>> CODETOOLS-7903756 will progress.
>> 
>> -l now honours also verbose mode, in which it prints not just matching
>> tests but all really run tests, and thus enabling much more easy
>> determining of all tests
>> 
>> help adjusted.
>> 
>> Maybe I'm missing plain quick initial all tests metod now, but with 
>> artificial -c MAX it seems doing exactly that
>
> Jiří Vaněk has updated the pull request incrementally with one additional commit since the last revision:
> 
>   replaced long string concatenation with StringBuilder

looks good to me, ready to be merged

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

Marked as reviewed by matcdac at github.com (no known OpenJDK username).

PR Review: https://git.openjdk.org/jcstress/pull/149#pullrequestreview-2158290785


More information about the jcstress-dev mailing list