Re: RFR: 8347173: java/net/DatagramSocket/InterruptibleDatagramSocket.java fails with virtual thread factory [v2]

2025-01-13 Thread Leonid Mesnik
On Mon, 13 Jan 2025 14:42:04 GMT, Jaikiran Pai wrote: >> Can I please get a review for this test-only change which proposes to skip >> this test when the `main()` is launched through a virtual thread? >> >> This test was introduced in https://bugs.openjdk.org/browse/JDK-8233018 , >> before vir

RFR: 8347840: Fix testlibrary compilation warnings

2025-01-15 Thread Leonid Mesnik
There few compiler warning disabled in the testlibary build. They should be fixed or localized and removed from build to prevent new possible issues. The main goal is to avoid new such issues in the testlibrary. Tested with tier1-5 to ensure that all tests were passed. - Commit mes

Re: RFR: 8347840: Fix testlibrary compilation warnings [v2]

2025-01-16 Thread Leonid Mesnik
> There few compiler warning disabled in the testlibary build. > They should be fixed or localized and removed from build to prevent new > possible issues. > > The main goal is to avoid new such issues in the testlibrary. > Tested with tier1-5 to ensure that all tests we

Re: RFR: 8347840: Fix testlibrary compilation warnings [v3]

2025-01-16 Thread Leonid Mesnik
> There few compiler warning disabled in the testlibary build. > They should be fixed or localized and removed from build to prevent new > possible issues. > > The main goal is to avoid new such issues in the testlibrary. > Tested with tier1-5 to ensure that all tests we

Re: RFR: 8347840: Fix testlibrary compilation warnings [v3]

2025-01-16 Thread Leonid Mesnik
On Thu, 16 Jan 2025 05:31:44 GMT, David Holmes wrote: >> Leonid Mesnik has updated the pull request incrementally with one additional >> commit since the last revision: >> >> revert change > > test/lib/jdk/test/lib/format/ArrayDiff.java line 110: > &g

Re: RFR: 8347840: Fix testlibrary compilation warnings [v3]

2025-01-16 Thread Leonid Mesnik
On Thu, 16 Jan 2025 07:09:46 GMT, Alan Bateman wrote: >> Leonid Mesnik has updated the pull request incrementally with one additional >> commit since the last revision: >> >> revert change > > test/lib/jdk/test/lib/thread/VThreadPinner.java line 97: > >

Re: RFR: 8347840: Fix testlibrary compilation warnings [v3]

2025-01-16 Thread Leonid Mesnik
On Thu, 16 Jan 2025 18:14:39 GMT, Leonid Mesnik wrote: >> There few compiler warning disabled in the testlibary build. >> They should be fixed or localized and removed from build to prevent new >> possible issues. >> >> The main goal is to avoid new such issues

Integrated: 8347840: Fix testlibrary compilation warnings

2025-01-17 Thread Leonid Mesnik
On Wed, 15 Jan 2025 23:48:33 GMT, Leonid Mesnik wrote: > There few compiler warning disabled in the testlibary build. > They should be fixed or localized and removed from build to prevent new > possible issues. > > The main goal is to avoid new such issues in the testlibrary.

Re: RFR: 8356171: Increase timeout for testcases as preparation for change of default timeout factor

2025-05-08 Thread Leonid Mesnik
On Thu, 8 May 2025 16:43:10 GMT, Leo Korinth wrote: >> This change tries to add timeout to individual testcases so that I am able >> to run them with a timeout factor of 1 in the future (JDK-8260555). >> >> The first commit changes the timeout factor to 0.7, so that I can run tests >> and test

Re: RFR: 8356171: Increase timeout for testcases as preparation for change of default timeout factor

2025-05-08 Thread Leonid Mesnik
On Thu, 8 May 2025 17:03:03 GMT, Leo Korinth wrote: >> doc/testing.md line 385: >> >>> 383: (`-timeoutFactor`). Also, some test cases that programmatically wait a >>> 384: certain amount of time will apply this factor. If we run in >>> 385: interpreted mode (`-Xcomp`), [RunTest.gmk](../make/RunT