You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ry...@apache.org on 2016/03/25 21:50:53 UTC
lucene-solr:branch_6x: SOLR-8902: fix glob test (put back the
fields.clear())
Repository: lucene-solr
Updated Branches:
refs/heads/branch_6x f785d2a03 -> e26c0b712
SOLR-8902: fix glob test (put back the fields.clear())
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/e26c0b71
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/e26c0b71
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/e26c0b71
Branch: refs/heads/branch_6x
Commit: e26c0b7125903e24e5865b825f0f9e993eb10178
Parents: f785d2a
Author: Ryan McKinley <ry...@apache.org>
Authored: Fri Mar 25 13:49:25 2016 -0700
Committer: Ryan McKinley <ry...@apache.org>
Committed: Fri Mar 25 13:49:54 2016 -0700
----------------------------------------------------------------------
.../src/java/org/apache/solr/search/SolrReturnFields.java | 7 +++++++
1 file changed, 7 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/e26c0b71/solr/core/src/java/org/apache/solr/search/SolrReturnFields.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/search/SolrReturnFields.java b/solr/core/src/java/org/apache/solr/search/SolrReturnFields.java
index b667f0e..34ef79e 100644
--- a/solr/core/src/java/org/apache/solr/search/SolrReturnFields.java
+++ b/solr/core/src/java/org/apache/solr/search/SolrReturnFields.java
@@ -130,6 +130,13 @@ public class SolrReturnFields extends ReturnFields {
}
augmenters.addTransformer( new RenameFieldTransformer( from, to, copy ) );
}
+ if( !_wantsAllFields && !globs.isEmpty() ) {
+ // TODO??? need to fill up the fields with matching field names in the index
+ // and add them to okFieldNames?
+ // maybe just get all fields?
+ // this would disable field selection optimization... i think thatis OK
+ fields.clear(); // this will get all fields, and use wantsField to limit
+ }
if( augmenters.size() == 1 ) {
transformer = augmenters.getTransformer(0);