You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by ib...@apache.org on 2021/08/10 06:08:44 UTC

[ignite] branch master updated: IGNITE-15273 Fixed triple flushing of meta information at the checkpoint. (#9308)

This is an automated email from the ASF dual-hosted git repository.

ibessonov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ignite.git


The following commit(s) were added to refs/heads/master by this push:
     new 02ae7c5  IGNITE-15273 Fixed triple flushing of meta information at the checkpoint. (#9308)
02ae7c5 is described below

commit 02ae7c5a35d579ced64bc5fd4bee35f2e576dd5f
Author: Kirill Tkalenko <tk...@yandex.ru>
AuthorDate: Tue Aug 10 09:08:23 2021 +0300

    IGNITE-15273 Fixed triple flushing of meta information at the checkpoint. (#9308)
---
 .../processors/cache/persistence/GridCacheOffheapManager.java      | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/GridCacheOffheapManager.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/GridCacheOffheapManager.java
index e1cdf2a..168c151 100644
--- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/GridCacheOffheapManager.java
+++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/GridCacheOffheapManager.java
@@ -267,15 +267,12 @@ public class GridCacheOffheapManager extends IgniteCacheOffheapManagerImpl imple
         assert grp.dataRegion().pageMemory() instanceof PageMemoryEx;
 
         syncMetadata(ctx);
-
-        // Double flushing memory buckets for decrease a time on write lock.
-        syncMetadata(ctx, ctx.executor(), false);
     }
 
     /** {@inheritDoc} */
     @Override public void beforeCheckpointBegin(Context ctx) throws IgniteCheckedException {
-        if (!ctx.nextSnapshot())
-            syncMetadata(ctx);
+        // Optimization: reducing the holding time of checkpoint write lock.
+        syncMetadata(ctx, ctx.executor(), false);
     }
 
     /** {@inheritDoc} */