You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by dw...@apache.org on 2021/01/20 18:05:41 UTC

[lucene-solr] branch master updated: LUCENE-9683: fix incorrect logic for measuring stall time information. (#2225)

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

dweiss pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git


The following commit(s) were added to refs/heads/master by this push:
     new 4da8f08  LUCENE-9683: fix incorrect logic for measuring stall time information. (#2225)
4da8f08 is described below

commit 4da8f08c63af789ea3ac31cf288aeae4381919a8
Author: Dawid Weiss <da...@carrotsearch.com>
AuthorDate: Wed Jan 20 19:05:22 2021 +0100

    LUCENE-9683: fix incorrect logic for measuring stall time information. (#2225)
---
 .../java/org/apache/lucene/index/ConcurrentMergeScheduler.java    | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/lucene/core/src/java/org/apache/lucene/index/ConcurrentMergeScheduler.java b/lucene/core/src/java/org/apache/lucene/index/ConcurrentMergeScheduler.java
index 34c9a74..cfa5d11 100644
--- a/lucene/core/src/java/org/apache/lucene/index/ConcurrentMergeScheduler.java
+++ b/lucene/core/src/java/org/apache/lucene/index/ConcurrentMergeScheduler.java
@@ -598,10 +598,12 @@ public class ConcurrentMergeScheduler extends MergeScheduler {
         return false;
       }
 
-      if (verbose() && startStallTime == 0) {
-        message("    too many merges; stalling...");
+      if (startStallTime == 0) {
+        startStallTime = System.currentTimeMillis();
+        if (verbose()) {
+          message("    too many merges; stalling...");
+        }
       }
-      startStallTime = System.currentTimeMillis();
       doStall();
     }