RFR: 8321474: TestAutoCreateSharedArchiveUpgrade.java should be updated with JDK 21 [v2]

David Holmes dholmes at openjdk.org
Thu Dec 7 22:11:15 UTC 2023


On Thu, 7 Dec 2023 21:26:51 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Matias Saavedra Silva has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fixed check for invalid JDK version
>
> test/hotspot/jtreg/runtime/cds/appcds/dynamicArchive/TestAutoCreateSharedArchiveUpgrade.java line 94:
> 
>> 92:                 doTest();
>> 93:             } catch (RuntimeException e) {
>> 94:                 throw new RuntimeException("Invalid AutoCreateSharedArchive JDK version: " + versions[i]);
> 
> If catching a general exception here you should now add that as the cause of the new one.

But why did you change this anyway?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17005#discussion_r1419712303


More information about the hotspot-runtime-dev mailing list