Re: RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java [v4]

2023-09-14 Thread Soumadipta Roy
On Thu, 14 Sep 2023 14:05:36 GMT, Martin Buchholz wrote: >> Soumadipta Roy has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - Changing test id name from security-manager-allow to security-manager >> - Creating a separate parallelized tes

Re: RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java [v4]

2023-09-14 Thread Aleksey Shipilev
On Thu, 14 Sep 2023 08:08:18 GMT, Soumadipta Roy wrote: >> This test is running in tier1, and takes about 400 seconds to complete. >> Thus, it drags the execution time of tier1 on large machines. The commit >> includes changing the sequential run of test cases in >> java/util/concurrent/tck/JS

Re: RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java [v4]

2023-09-14 Thread Martin Buchholz
On Thu, 14 Sep 2023 08:08:18 GMT, Soumadipta Roy wrote: >> This test is running in tier1, and takes about 400 seconds to complete. >> Thus, it drags the execution time of tier1 on large machines. The commit >> includes changing the sequential run of test cases in >> java/util/concurrent/tck/JS

Re: RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java [v3]

2023-09-14 Thread Soumadipta Roy
On Thu, 14 Sep 2023 04:29:17 GMT, Martin Buchholz wrote: >> Soumadipta Roy has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Adding summary for each parallelized section. > > Marked as reviewed by martin (Reviewer). Hi @Martin-Buchholz I

Re: RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java [v4]

2023-09-14 Thread Soumadipta Roy
> This test is running in tier1, and takes about 400 seconds to complete. Thus, > it drags the execution time of tier1 on large machines. The commit includes > changing the sequential run of test cases in > java/util/concurrent/tck/JSR166TestCase.java to the best possible combination > of paral

Re: RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java [v3]

2023-09-13 Thread Martin Buchholz
On Wed, 13 Sep 2023 12:27:39 GMT, Soumadipta Roy wrote: >> This test is running in tier1, and takes about 400 seconds to complete. >> Thus, it drags the execution time of tier1 on large machines. The commit >> includes changing the sequential run of test cases in >> java/util/concurrent/tck/JS

Re: RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java [v2]

2023-09-13 Thread Martin Buchholz
On Thu, 7 Sep 2023 16:22:18 GMT, Soumadipta Roy wrote: >> test/jdk/java/util/concurrent/tck/JSR166TestCase.java line 45: >> >>> 43: * @modules java.management >>> 44: * @run junit/othervm/timeout=1000 JSR166TestCase >>> 45: * @run junit/othervm/timeout=1000 -Djava.security.manager=allow >>>

Re: RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java [v2]

2023-09-13 Thread Soumadipta Roy
On Thu, 7 Sep 2023 16:08:16 GMT, Martin Buchholz wrote: > Hello Soumadipta - pleased to "meet" you. > > If the intent is to split the tests into relatively more and less important > variants for placing into separate tiers, that should be clear from the > @summary's in the tests. Hi @Martin-B

Re: RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java [v3]

2023-09-13 Thread Soumadipta Roy
> This test is running in tier1, and takes about 400 seconds to complete. Thus, > it drags the execution time of tier1 on large machines. The commit includes > changing the sequential run of test cases in > java/util/concurrent/tck/JSR166TestCase.java to the best possible combination > of paral

Re: RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java [v2]

2023-09-07 Thread Soumadipta Roy
On Thu, 7 Sep 2023 15:58:26 GMT, Martin Buchholz wrote: >> Soumadipta Roy has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Remove extra new line > > test/jdk/java/util/concurrent/tck/JSR166TestCase.java line 45: > >> 43: * @modules java

Re: RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java [v2]

2023-09-07 Thread Martin Buchholz
On Thu, 7 Sep 2023 14:10:20 GMT, Soumadipta Roy wrote: >> This test is running in tier1, and takes about 400 seconds to complete. >> Thus, it drags the execution time of tier1 on large machines. The commit >> includes changing the sequential run of test cases in >> java/util/concurrent/tck/JSR

Re: RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java [v2]

2023-09-07 Thread Martin Buchholz
On Thu, 7 Sep 2023 14:10:20 GMT, Soumadipta Roy wrote: >> This test is running in tier1, and takes about 400 seconds to complete. >> Thus, it drags the execution time of tier1 on large machines. The commit >> includes changing the sequential run of test cases in >> java/util/concurrent/tck/JSR

RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java

2023-09-07 Thread Soumadipta Roy
This test is running in tier1, and takes about 400 seconds to complete. Thus, it drags the execution time of tier1 on large machines. The commit includes changing the sequential run of test cases in java/util/concurrent/tck/JSR166TestCase.java to the best possible combination of parallelization

Re: RFR: 8315683: Parallelize java/util/concurrent/tck/JSR166TestCase.java [v2]

2023-09-07 Thread Soumadipta Roy
> This test is running in tier1, and takes about 400 seconds to complete. Thus, > it drags the execution time of tier1 on large machines. The commit includes > changing the sequential run of test cases in > java/util/concurrent/tck/JSR166TestCase.java to the best possible combination > of paral