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 2013/01/16 00:55:04 UTC
svn commit: r1433748 -
/lucene/dev/branches/lucene4547/lucene/core/src/java/org/apache/lucene/search/FieldCacheImpl.java
Author: mikemccand
Date: Tue Jan 15 23:55:03 2013
New Revision: 1433748
URL: http://svn.apache.org/viewvc?rev=1433748&view=rev
Log:
fix TestSort
Modified:
lucene/dev/branches/lucene4547/lucene/core/src/java/org/apache/lucene/search/FieldCacheImpl.java
Modified: lucene/dev/branches/lucene4547/lucene/core/src/java/org/apache/lucene/search/FieldCacheImpl.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene4547/lucene/core/src/java/org/apache/lucene/search/FieldCacheImpl.java?rev=1433748&r1=1433747&r2=1433748&view=diff
==============================================================================
--- lucene/dev/branches/lucene4547/lucene/core/src/java/org/apache/lucene/search/FieldCacheImpl.java (original)
+++ lucene/dev/branches/lucene4547/lucene/core/src/java/org/apache/lucene/search/FieldCacheImpl.java Tue Jan 15 23:55:03 2013
@@ -1274,9 +1274,10 @@ class FieldCacheImpl implements FieldCac
throws IOException {
BinaryDocValues valuesIn = reader.getBinaryDocValues(key.field);
-
- // TODO: would be nice to fallback to SortedDV if it's
- // available but BinaryDV isn't?
+ if (valuesIn == null) {
+ // nocommit is this auto-fallback ... OK?
+ valuesIn = reader.getSortedDocValues(key.field);
+ }
if (valuesIn != null) {
return valuesIn;