RFR: 7903750: TimeBudget (-tb) does not fulfill its promisses [v6]

Galder Zamarreño galder at openjdk.org
Wed Mar 26 08:45:29 UTC 2025


On Mon, 24 Mar 2025 18:18:54 GMT, Jiří Vaněk <jvanek at openjdk.org> wrote:

>> Added warning, when time budget is to small, and final run will run at least two times longer.
>
> Jiří Vaněk has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - directly return new instance without intermediate variable
>    
>    Co-authored-by: Galder Zamarreño <galder at redhat.com>
>  - Removed table like formatting for time budget warnning

Changes requested by galder (no project role).

jcstress-core/src/main/java/org/openjdk/jcstress/JCStress.java line 89:

> 87: 
> 88:     private TimeBudget getTimeBudget(ConfigsWithScheduler config) {
> 89:         TimeBudget timeBudget = new TimeBudget(config.configs.size(), opts.timeBudget());

Remove this line, not necessary now.

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

PR Review: https://git.openjdk.org/jcstress/pull/161#pullrequestreview-2716299829
PR Review Comment: https://git.openjdk.org/jcstress/pull/161#discussion_r2013633778


More information about the jcstress-dev mailing list