[crac] RFR: 8369566: CRaC: Record metrics during checkpoint [v3]
Radim Vansa
rvansa at openjdk.org
Tue Oct 21 06:31:41 UTC 2025
On Thu, 16 Oct 2025 12:50:09 GMT, Timofei Pushkin <tpushkin at openjdk.org> wrote:
>> Radim Vansa has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Fix for PathPatternTest
>
> src/java.base/share/classes/jdk/internal/crac/Score.java line 60:
>
>> 58: * recording metadata this is ignored.
>> 59: * On checkpoint the metrics are not reset; if that is desired invoke {@link #resetAll()}
>> 60: * manually.
>
> BTW we should also be documenting this on the engine API level: whether something is reset on checkpoint or not. AFAIR user data is reset, constraints are not, scores — I haven't checked yet. This can be postponed until we claim the support of repeated checkpoints, I am just saying.
Good point. I think that we're not resetting on the engine level ATM, but we should: otherwise the Java-space `resetAll()` is not effective.
-------------
PR Review Comment: https://git.openjdk.org/crac/pull/269#discussion_r2446849467
More information about the crac-dev
mailing list