[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