You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by mi...@apache.org on 2014/10/23 17:05:32 UTC
svn commit: r1633833 - in /lucene/dev/branches/branch_5x: ./ lucene/
lucene/core/
lucene/core/src/java/org/apache/lucene/index/DefaultIndexingChain.java
Author: mikemccand
Date: Thu Oct 23 15:05:32 2014
New Revision: 1633833
URL: http://svn.apache.org/r1633833
Log:
LUCENE-6019: add more safety during DV flush
Modified:
lucene/dev/branches/branch_5x/ (props changed)
lucene/dev/branches/branch_5x/lucene/ (props changed)
lucene/dev/branches/branch_5x/lucene/core/ (props changed)
lucene/dev/branches/branch_5x/lucene/core/src/java/org/apache/lucene/index/DefaultIndexingChain.java
Modified: lucene/dev/branches/branch_5x/lucene/core/src/java/org/apache/lucene/index/DefaultIndexingChain.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_5x/lucene/core/src/java/org/apache/lucene/index/DefaultIndexingChain.java?rev=1633833&r1=1633832&r2=1633833&view=diff
==============================================================================
--- lucene/dev/branches/branch_5x/lucene/core/src/java/org/apache/lucene/index/DefaultIndexingChain.java (original)
+++ lucene/dev/branches/branch_5x/lucene/core/src/java/org/apache/lucene/index/DefaultIndexingChain.java Thu Oct 23 15:05:32 2014
@@ -130,6 +130,10 @@ final class DefaultIndexingChain extends
PerField perField = fieldHash[i];
while (perField != null) {
if (perField.docValuesWriter != null) {
+ if (perField.fieldInfo.hasDocValues() == false) {
+ // BUG
+ throw new AssertionError("segment=" + state.segmentInfo + ": field=\"" + perField.fieldInfo.name + "\" has no docValues but wrote them");
+ }
if (dvConsumer == null) {
// lazy init
DocValuesFormat fmt = state.segmentInfo.getCodec().docValuesFormat();
@@ -139,6 +143,9 @@ final class DefaultIndexingChain extends
perField.docValuesWriter.finish(docCount);
perField.docValuesWriter.flush(state, dvConsumer);
perField.docValuesWriter = null;
+ } else if (perField.fieldInfo.hasDocValues()) {
+ // BUG
+ throw new AssertionError("segment=" + state.segmentInfo + ": field=\"" + perField.fieldInfo.name + "\" has docValues but did not write them");
}
perField = perField.next;
}
@@ -156,6 +163,16 @@ final class DefaultIndexingChain extends
IOUtils.closeWhileHandlingException(dvConsumer);
}
}
+
+ if (state.fieldInfos.hasDocValues() == false) {
+ if (dvConsumer != null) {
+ // BUG
+ throw new AssertionError("segment=" + state.segmentInfo + ": fieldInfos has no docValues but wrote them");
+ }
+ } else if (dvConsumer == null) {
+ // BUG
+ throw new AssertionError("segment=" + state.segmentInfo + ": fieldInfos has docValues but did not wrote them");
+ }
}
/** Catch up for all docs before us that had no stored