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:13:09 UTC

[lucene-solr] branch branch_8x 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 branch_8x
in repository https://gitbox.apache.org/repos/asf/lucene-solr.git


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

commit 2ca48ac08079c35bd4e4efde881c56db7fbe229f
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 cece582..fe90b38 100644
--- a/lucene/core/src/java/org/apache/lucene/index/ConcurrentMergeScheduler.java
+++ b/lucene/core/src/java/org/apache/lucene/index/ConcurrentMergeScheduler.java
@@ -595,10 +595,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();
     }