You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Alexey Goncharuk (Jira)" <ji...@apache.org> on 2020/04/06 10:54:00 UTC

[jira] [Updated] (IGNITE-12802) Move checkpoint state fields to CheckpointProgress

     [ https://issues.apache.org/jira/browse/IGNITE-12802?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alexey Goncharuk updated IGNITE-12802:
--------------------------------------
    Fix Version/s: 2.9

> Move checkpoint state fields to CheckpointProgress
> --------------------------------------------------
>
>                 Key: IGNITE-12802
>                 URL: https://issues.apache.org/jira/browse/IGNITE-12802
>             Project: Ignite
>          Issue Type: Improvement
>            Reporter: Alexey Goncharuk
>            Assignee: Stanilovsky Evgeny
>            Priority: Major
>             Fix For: 2.9
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> This is a review follow-up for IGNITE-7792. I've noticed that quite a few fields in {{GridCacheDatabaseSharedManager}} are related to the state of current checkpoint:
> {code}
> writtenPagesCntr
> syncedPagesCntr
> evictedPagesCntr
> currCheckpointPagesCnt
> {code}
> After checkpoint is completed, these fields are reset. On the other hand, we have a separate class to track the state of current checkpoint: {{CheckpointProgressImpl}}. I believe it makes sense to move these fields to the separate class. Perhaps, it also makes sense to make this class a top-level class.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)