You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by si...@apache.org on 2020/06/26 13:19:21 UTC

[lucene-solr] 01/02: also remove merge from pending

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

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

commit 56351688673a7ce35cfa57a3bbdf9bb53a79e3b9
Author: Simon Willnauer <si...@apache.org>
AuthorDate: Fri Jun 26 15:18:53 2020 +0200

    also remove merge from pending
---
 lucene/core/src/java/org/apache/lucene/index/IndexWriter.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/lucene/core/src/java/org/apache/lucene/index/IndexWriter.java b/lucene/core/src/java/org/apache/lucene/index/IndexWriter.java
index c7e9f90..0fc058f 100644
--- a/lucene/core/src/java/org/apache/lucene/index/IndexWriter.java
+++ b/lucene/core/src/java/org/apache/lucene/index/IndexWriter.java
@@ -3387,9 +3387,11 @@ public class IndexWriter implements Closeable, TwoPhaseCommit, Accountable,
         if (closeReaders) {
           IOUtils.applyToAll(onCommitMerges.merges, merge -> {
             // that merge is broken we need to clean up after it - it's fine we still have the IW lock to do this
-            assert pendingMerges.contains(merge) : "merge should be pending but isn't: " + merge.segString();
+            boolean removed = pendingMerges.remove(merge);
+            assert removed: "merge should be pending but isn't: " + merge.segString();
             abortOneMerge(merge);
             mergeFinish(merge);
+
           });
         }
       }