Apache
/
lucene
mirror of https://github.com/apache/lucene.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

clear nocommits 

git-svn-id: https://svn.apache.org/repos/asf/lucene/dev/branches/lucene4547@1442411 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Robert Muir 2013-02-04 22:41:51 +00:00
parent 9e0e90b996
commit fb7573adcc
2 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
lucene/core/src/java/org/apache/lucene/index/FieldInfo.java
View File

@ -234,7 +234,6 @@ public final class FieldInfo {
assert checkConsistency();
}
// nocommit type is always number? should we remove this?
void setNormValueType(DocValuesType type) {
if (normType != null && normType != type) {
throw new IllegalArgumentException("cannot change Norm type from " + normType + " to " + type + " for field \"" + name + "\"");

6
lucene/core/src/java/org/apache/lucene/index/NormsConsumerPerField.java
View File

@ -43,8 +43,6 @@ final class NormsConsumerPerField extends InvertedDocEndConsumerPerField impleme
void finish() throws IOException {
if (fieldInfo.isIndexed() && !fieldInfo.omitsNorms()) {
if (consumer == null) {
// nocommit wrongish? what about the
// only-doc-with-norms-enabled-hits-exc case?
fieldInfo.setNormValueType(FieldInfo.DocValuesType.NUMERIC);
consumer = new NumericDocValuesWriter(fieldInfo, docState.docWriter.bytesUsed);
}
@ -56,8 +54,8 @@ final class NormsConsumerPerField extends InvertedDocEndConsumerPerField impleme
int docCount = state.segmentInfo.getDocCount();
if (consumer == null) {
return; // null type - not omitted but not written -
// nocommit meaning the only docs that had
// norms hit exceptions?
// meaning the only docs that had
// norms hit exceptions (but indexed=true is set...)
}
consumer.finish(docCount);
consumer.flush(state, normsWriter);