You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by se...@apache.org on 2020/11/03 06:46:40 UTC

[ignite] branch master updated: IGNITE-13650 updateCurrentHandle method call is moved from under assert clause - Fixes #8412.

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

sergeychugunov 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 6eb8f74  IGNITE-13650 updateCurrentHandle method call is moved from under assert clause - Fixes #8412.
6eb8f74 is described below

commit 6eb8f7407499fc76a4284d981d8aea5c5de45833
Author: ktkalenko <kt...@gridgain.com>
AuthorDate: Tue Nov 3 09:45:30 2020 +0300

    IGNITE-13650 updateCurrentHandle method call is moved from under assert clause - Fixes #8412.
    
    Signed-off-by: Sergey Chugunov <se...@gmail.com>
---
 .../processors/cache/persistence/wal/FileWriteAheadLogManager.java    | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/wal/FileWriteAheadLogManager.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/wal/FileWriteAheadLogManager.java
index a92168b..ff64f5b 100644
--- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/wal/FileWriteAheadLogManager.java
+++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/wal/FileWriteAheadLogManager.java
@@ -1297,7 +1297,9 @@ public class FileWriteAheadLogManager extends GridCacheSharedManagerAdapter impl
             if (next.getSegmentId() - lashCheckpointFileIdx() >= maxSegCountWithoutCheckpoint)
                 cctx.database().forceCheckpoint("too big size of WAL without checkpoint");
 
-            assert updateCurrentHandle(next, hnd) : "Concurrent updates on rollover are not allowed";
+            boolean updated = updateCurrentHandle(next, hnd);
+
+            assert updated : "Concurrent updates on rollover are not allowed";
 
             if (walAutoArchiveAfterInactivity > 0)
                 lastRecordLoggedMs.set(0);