[crac] RFR: Make CheckpointException/RestoreException aggregate-only [v7]

Radim Vansa duke at openjdk.org
Thu Jun 8 11:50:36 UTC 2023


On Thu, 8 Jun 2023 09:23:27 GMT, Anton Kozlov <akozlov at openjdk.org> wrote:

>> Radim Vansa has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Moved exception after native failure
>
> src/java.base/share/classes/jdk/crac/CheckpointException.java line 36:
> 
>> 34: 
>> 35:     /**
>> 36:      * Creates a {@code CheckpointException}.
> 
> I probably misclicked "Add comment",
> 
> https://docs.oracle.com/en/java/javase/17/docs/api/java.base/java/lang/Throwable.html#%3Cinit%3E(java.lang.String,java.lang.Throwable,boolean,boolean):
>> Subclasses of Throwable should document any conditions under which suppression is disabled and document conditions under which the stack trace is not writable.
> 
> The javadoc should specify the stack trace is not writeable, here and for other exceptions

Updated. I've also made those classes final to prevent subclassing with a specific meaning - the actual information should be in the suppressed exceptions.

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

PR Review Comment: https://git.openjdk.org/crac/pull/64#discussion_r1222925408


More information about the crac-dev mailing list