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

svn commit: r1602140 - /lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/ReadersAndUpdates.java

Author: shaie
Date: Thu Jun 12 12:26:20 2014
New Revision: 1602140

URL: http://svn.apache.org/r1602140
Log:
add comments to clarify code

Modified:
    lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/ReadersAndUpdates.java

Modified: lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/ReadersAndUpdates.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/ReadersAndUpdates.java?rev=1602140&r1=1602139&r2=1602140&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/ReadersAndUpdates.java (original)
+++ lucene/dev/trunk/lucene/core/src/java/org/apache/lucene/index/ReadersAndUpdates.java Thu Jun 12 12:26:20 2014
@@ -185,11 +185,13 @@ class ReadersAndUpdates {
       getReader(context).decRef();
       assert reader != null;
     }
+    // force new liveDocs in initWritableLiveDocs even if it's null
     liveDocsShared = true;
     if (liveDocs != null) {
       return new SegmentReader(reader.getSegmentInfo(), reader, liveDocs, info.info.getDocCount() - info.getDelCount() - pendingDeleteCount);
     } else {
-      assert reader.getLiveDocs() == liveDocs;
+      // liveDocs == null and reader != null. That can only be if there are no deletes
+      assert reader.getLiveDocs() == null;
       reader.incRef();
       return reader;
     }