[crac] RFR: 8368570: [CRaC] Improve JdwpTransportTest [v2]
Roman Marchenko
rmarchenko at openjdk.org
Mon Sep 29 10:52:41 UTC 2025
On Mon, 29 Sep 2025 09:34:35 GMT, Timofei Pushkin <tpushkin at openjdk.org> wrote:
>> Roman Marchenko has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Update test/lib/jdk/test/lib/crac/CracProcess.java
>>
>> fix grammar
>>
>> Co-authored-by: Timofei Pushkin <pushkin.td at gmail.com>
>> - Update test/jdk/jdk/crac/jdwp/JdwpTransportTest.java
>>
>> fix copyright
>>
>> Co-authored-by: Timofei Pushkin <pushkin.td at gmail.com>
>
> test/lib/jdk/test/lib/crac/CracProcess.java line 236:
>
>> 234: }
>> 235:
>> 236: public static void dumpProcess(long pid) throws IOException, InterruptedException {
>
> `JdwpTransportTest` runs on all platforms and I believe this is not going to work on non-Linux
I think it should work on linuxes, macos, and cygwin. cygwin doesn't produce a core file of course. For other cases, the command will fail with a message. I don't think this is important as it will be triggered when process hangs only.
-------------
PR Review Comment: https://git.openjdk.org/crac/pull/265#discussion_r2387501097
More information about the crac-dev
mailing list