ignite-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Maxim Muzafarov (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (IGNITE-8957) testFailGetLock() constantly fails. Last entry checkpoint history can be empty
Date Mon, 09 Jul 2018 15:28:01 GMT

    [ https://issues.apache.org/jira/browse/IGNITE-8957?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16537066#comment-16537066
] 

Maxim Muzafarov commented on IGNITE-8957:
-----------------------------------------

[~andmed],

Minor comment to this PR:

I've also found that if we return {{`remove`}} array earlier we will miss all logic related
to {{chp.walFilesDeleted(deleted)}}. Here it would not be a problem because we have checks
{{`while (histMap.size() > maxCpHistMemSize)`}}.


 For me better to use if-else statements rather that `return` ealier.

> testFailGetLock() constantly fails. Last entry checkpoint history can be empty
> ------------------------------------------------------------------------------
>
>                 Key: IGNITE-8957
>                 URL: https://issues.apache.org/jira/browse/IGNITE-8957
>             Project: Ignite
>          Issue Type: Bug
>          Components: persistence
>    Affects Versions: 2.7
>            Reporter: Maxim Muzafarov
>            Assignee: Andrew Medvedev
>            Priority: Major
>              Labels: MakeTeamcityGreenAgain
>
> IgniteChangeGlobalStateTest#testFailGetLock constantly fails with exception:
> {code}
> java.lang.AssertionError
> 	at org.apache.ignite.internal.processors.cache.persistence.checkpoint.CheckpointHistory.onCheckpointFinished(CheckpointHistory.java:205)
> 	at org.apache.ignite.internal.processors.cache.persistence.GridCacheDatabaseSharedManager$Checkpointer.markCheckpointEnd(GridCacheDatabaseSharedManager.java:3654)
> 	at org.apache.ignite.internal.processors.cache.persistence.GridCacheDatabaseSharedManager$Checkpointer.doCheckpoint(GridCacheDatabaseSharedManager.java:3178)
> 	at org.apache.ignite.internal.processors.cache.persistence.GridCacheDatabaseSharedManager$Checkpointer.body(GridCacheDatabaseSharedManager.java:2953)
> 	at org.apache.ignite.internal.util.worker.GridWorker.run(GridWorker.java:110)
> 	at java.lang.Thread.run(Thread.java:748)
> {code}
> As Sergey Chugunov [mentioned|https://issues.apache.org/jira/browse/IGNITE-8737?focusedCommentId=16535062&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-16535062],
issue can be solved different ways:
> {quote}
> It seems we missed a case when lastEntry may be empty. We may choose here from two options:
> * Check if histMap is empty inside onCheckpointFinished. If it is just don't log anything
(it was the very first checkpoint).
> * Check in caller that there is no history, calculate necessary index in caller and pass
it to onCheckpointFinished to prepare correct log message.{quote}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message