On Tue, 15 Jul 2025 11:55:01 GMT, Matthew Donovan <mdono...@openjdk.org> wrote:

>> Mikhail Yankelevich has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   breaking the loop when interrupt flag is set
>
> test/jdk/sun/security/krb5/auto/KDC.java line 1649:
> 
>> 1647:         } else {
>> 1648:             try {
>> 1649:                 thread1.interrupt();
> 
> Each thread catches and ignores `Exception` which means the `interrupt()` 
> call is ineffective.  The threads either need to exit when they catch 
> `Exception` or explicitly catch `InterruptedException` and exit when it is 
> thrown.

Done in the next commit

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/26247#discussion_r2207756831

Reply via email to