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

Anton Kozlov akozlov at openjdk.org
Fri Jun 9 13:51:10 UTC 2023


On Thu, 8 Jun 2023 11:50:31 GMT, Radim Vansa <duke at openjdk.org> wrote:

>> CheckpointException and RestoreException (both jdk.crac and javax.crac) cannot carry any message, cause and don't collect stack trace. The sole purpose of these is to aggregate other exceptions in suppressed exceptions.
>
> Radim Vansa has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update javadoc, make final

src/java.base/share/classes/jdk/crac/Core.java line 89:

> 87:         if (codes.length == 0) {
> 88:             exception.addSuppressed(new RuntimeException("Native checkpoint failed."));
> 89:         }

Turns out this fires on checkpoint dry runs. Suppose a Resource has thrown an exception, we go to JVM just to check no native FD is open. When everything is OK on native side (codes and messages are empty), this exception is thrown, although no native checkpoint is attempted.

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

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


More information about the crac-dev mailing list