Re: RFR: 8260555: Change the default TIMEOUT_FACTOR from 4 to 1 [v2]

2025-08-13 Thread Leo Korinth
On Wed, 13 Aug 2025 09:44:33 GMT, Andrey Turbanov wrote: >> Leo Korinth has updated the pull request incrementally with one additional >> commit since the last revision: >> >> After suggestions from Erik and Andrey > > test/langtools/tools/lib/toolbox/ToolBox.java line 480: > >> 478: >> 479

Re: RFR: 8260555: Change the default TIMEOUT_FACTOR from 4 to 1 [v2]

2025-08-13 Thread Leo Korinth
On Wed, 13 Aug 2025 13:25:48 GMT, Erik Joelsson wrote: >> Leo Korinth has updated the pull request incrementally with one additional >> commit since the last revision: >> >> After suggestions from Erik and Andrey > > make/RunTests.gmk line 939: > >> 937: >> 938: JTREG_AUTO_PROBLEM_LISTS :

Re: RFR: 8260555: Change the default TIMEOUT_FACTOR from 4 to 1 [v2]

2025-08-13 Thread Leo Korinth
> This changes the timeout factor from 4 to 1. Most of the changes add timeouts > to individual test cases so that I am able to run them with a timeout factor > of 0.7 (some margin to the checked in factor of one) > > In addition to changing the timeout factor, I am also using a library call to

Re: RFR: 8260555: Change the default TIMEOUT_FACTOR from 4 to 1

2025-08-13 Thread Leo Korinth
On Wed, 13 Aug 2025 07:26:59 GMT, Serguei Spitsyn wrote: > I've reviewed the Serviceability related tweaks and I'm okay with them in > general. But I'm curious if you do not see any timeouts with this anymore. I only got the four timeouts described in the description, I got a few other failure

Re: RFR: 8260555: Change the default TIMEOUT_FACTOR from 4 to 1

2025-08-13 Thread Leo Korinth
On Wed, 13 Aug 2025 01:51:44 GMT, SendaoYan wrote: >> This changes the timeout factor from 4 to 1. Most of the changes add >> timeouts to individual test cases so that I am able to run them with a >> timeout factor of 0.7 (some margin to the checked in factor of one) >> >> In addition to chang

Re: RFR: 8260555: Change the default TIMEOUT_FACTOR from 4 to 1

2025-08-13 Thread Erik Joelsson
On Tue, 12 Aug 2025 17:01:41 GMT, Leo Korinth wrote: > This changes the timeout factor from 4 to 1. Most of the changes add timeouts > to individual test cases so that I am able to run them with a timeout factor > of 0.7 (some margin to the checked in factor of one) > > In addition to changing

Re: RFR: 8260555: Change the default TIMEOUT_FACTOR from 4 to 1

2025-08-13 Thread Leo Korinth
On Tue, 12 Aug 2025 17:52:02 GMT, Chris Plummer wrote: > > sun/tools/jhsdb/BasicLauncherTest.java > > I am unsure about this one, it has not failed on my runs before, even with > > a timeout factor of 0.7, maybe I was unlucky. > > @lkorinth Can you send me a link to the failure? I sent it to y

Re: RFR: 8260555: Change the default TIMEOUT_FACTOR from 4 to 1

2025-08-13 Thread Leo Korinth
On Wed, 13 Aug 2025 01:47:43 GMT, SendaoYan wrote: >> This changes the timeout factor from 4 to 1. Most of the changes add >> timeouts to individual test cases so that I am able to run them with a >> timeout factor of 0.7 (some margin to the checked in factor of one) >> >> In addition to chang

Re: RFR: 8260555: Change the default TIMEOUT_FACTOR from 4 to 1

2025-08-13 Thread Andrey Turbanov
On Tue, 12 Aug 2025 17:01:41 GMT, Leo Korinth wrote: > This changes the timeout factor from 4 to 1. Most of the changes add timeouts > to individual test cases so that I am able to run them with a timeout factor > of 0.7 (some margin to the checked in factor of one) > > In addition to changing

Re: RFR: 8260555: Change the default TIMEOUT_FACTOR from 4 to 1

2025-08-13 Thread Serguei Spitsyn
On Tue, 12 Aug 2025 17:01:41 GMT, Leo Korinth wrote: > This changes the timeout factor from 4 to 1. Most of the changes add timeouts > to individual test cases so that I am able to run them with a timeout factor > of 0.7 (some margin to the checked in factor of one) > > In addition to changing