[crac] RFR: Fix ordering of invocation on Resources [v10]
Anton Kozlov
akozlov at openjdk.org
Wed May 10 15:25:56 UTC 2023
On Wed, 10 May 2023 13:48:08 GMT, Radim Vansa <duke at openjdk.org> wrote:
>> It seems we should have allowed the message in the CheckpointException, to avoid this problem. The only place where CheckpointException with the message is called is recursive checkpoint detection from https://github.com/openjdk/crac/pull/6.
>>
>> OK for this PR.
>
> That's not entirely true; CheckpointException is a public API and Resources can use it. In fact I find it quite natural to use for reporting any 'generic' error during checkpoint; if we want to limit usage to internal we need to make the constructors package-private constructors.
Hiding all constructors makes sense. I think it will be helpful to have CheckpointException, CheckpointResourceExceptions and descedants something formal, something that can be programmatically queried for the problem. I propose to investigate the possiblity in #64 which touches related pieces.
-------------
PR Review Comment: https://git.openjdk.org/crac/pull/60#discussion_r1190070472
More information about the crac-dev
mailing list