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 2011/11/12 19:18:51 UTC
svn commit: r1201296 -
/lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/util/FieldCacheSanityChecker.java
Author: mikemccand
Date: Sat Nov 12 18:18:51 2011
New Revision: 1201296
URL: http://svn.apache.org/viewvc?rev=1201296&view=rev
Log:
LUCENE-3443: if a field has both Bits and array entry, it's not insane (Bits is the DocsWithField)
Modified:
lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/util/FieldCacheSanityChecker.java
Modified: lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/util/FieldCacheSanityChecker.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/util/FieldCacheSanityChecker.java?rev=1201296&r1=1201295&r2=1201296&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/util/FieldCacheSanityChecker.java (original)
+++ lucene/dev/branches/branch_3x/lucene/src/java/org/apache/lucene/util/FieldCacheSanityChecker.java Sat Nov 12 18:18:51 2011
@@ -119,6 +119,13 @@ public final class FieldCacheSanityCheck
final CacheEntry item = cacheEntries[i];
final Object val = item.getValue();
+ // It's OK to have dup entries, where one is eg
+ // float[] and the other is the Bits (from
+ // getDocWithField())
+ if (val instanceof Bits) {
+ continue;
+ }
+
if (val instanceof FieldCache.CreationPlaceholder)
continue;