You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@ignite.apache.org by GitBox <gi...@apache.org> on 2020/07/23 18:58:11 UTC

[GitHub] [ignite] zstan commented on a change in pull request #8072: IGNITE-13289 Fix failure of PagesWriteThrottleSmokeTest.testThrottle

zstan commented on a change in pull request #8072:
URL: https://github.com/apache/ignite/pull/8072#discussion_r459662497



##########
File path: modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/GridCacheDatabaseSharedManager.java
##########
@@ -4270,6 +4270,8 @@ private long updateLastCheckpointTime() {
          */
         private void markCheckpointEnd(Checkpoint chp) throws IgniteCheckedException {
             synchronized (this) {
+                getCheckpointer().currentProgress().clearCounters();

Review comment:
       Uff, i know about mentioned ticket, it looks like huge and controversial, but ok, i agree to change it, just because now it looks strange :)




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org