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/24 20:11:36 UTC

[lucene-solr] branch jira/lucene-8962 updated: remove TODO

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


The following commit(s) were added to refs/heads/jira/lucene-8962 by this push:
     new 827334d  remove TODO
827334d is described below

commit 827334deef23236884c76deeda17e38f1d950725
Author: Simon Willnauer <si...@apache.org>
AuthorDate: Wed Jun 24 22:11:19 2020 +0200

    remove TODO
---
 lucene/core/src/java/org/apache/lucene/index/IndexWriter.java | 1 -
 1 file changed, 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 a411ce8..4eed9d8 100644
--- a/lucene/core/src/java/org/apache/lucene/index/IndexWriter.java
+++ b/lucene/core/src/java/org/apache/lucene/index/IndexWriter.java
@@ -3298,7 +3298,6 @@ public class IndexWriter implements Closeable, TwoPhaseCommit, Accountable,
                 ), MergeTrigger.COMMIT, UNBOUNDED_MAX_MERGE_SEGMENTS);
                 if (onCommitMerges != null) {
                   for (MergePolicy.OneMerge merge : onCommitMerges.merges) {
-                    // TODO we need to release these readers in the case of an aborted merge
                     merge.initMergeReaders(IOContext.DEFAULT, sci -> getPooledInstance(sci, true));
                   }
                 }