You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by cp...@apache.org on 2018/02/09 17:11:13 UTC
lucene-solr:branch_7x: SOLR-11349: Rename ResponseBuilder's
getQueryCommand to createQueryCommand.
Repository: lucene-solr
Updated Branches:
refs/heads/branch_7x 8bb135aed -> b8224ad6b
SOLR-11349: Rename ResponseBuilder's getQueryCommand to createQueryCommand.
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/b8224ad6
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/b8224ad6
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/b8224ad6
Branch: refs/heads/branch_7x
Commit: b8224ad6ba615b1eb0f6244eb4ea0b85981d84d6
Parents: 8bb135a
Author: Christine Poerschke <cp...@apache.org>
Authored: Fri Feb 9 16:47:36 2018 +0000
Committer: Christine Poerschke <cp...@apache.org>
Committed: Fri Feb 9 17:10:16 2018 +0000
----------------------------------------------------------------------
solr/CHANGES.txt | 2 ++
.../org/apache/solr/handler/component/QueryComponent.java | 2 +-
.../org/apache/solr/handler/component/ResponseBuilder.java | 7 ++++++-
solr/core/src/java/org/apache/solr/request/SimpleFacets.java | 2 +-
4 files changed, 10 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/b8224ad6/solr/CHANGES.txt
----------------------------------------------------------------------
diff --git a/solr/CHANGES.txt b/solr/CHANGES.txt
index f4ea68f..e6a54a0 100644
--- a/solr/CHANGES.txt
+++ b/solr/CHANGES.txt
@@ -248,6 +248,8 @@ Other Changes
* SOLR-11933: Make DIH UI page safer by not default checking the clean checkbox (Eric Pugh via Tomás Fernández Löbbe)
+* SOLR-11349: Rename ResponseBuilder's getQueryCommand to createQueryCommand. (Christine Poerschke)
+
================== 7.2.1 ==================
Consult the LUCENE_CHANGES.txt file for additional, low level, changes in this release.
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/b8224ad6/solr/core/src/java/org/apache/solr/handler/component/QueryComponent.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/handler/component/QueryComponent.java b/solr/core/src/java/org/apache/solr/handler/component/QueryComponent.java
index ec3c79b..dfa2079 100644
--- a/solr/core/src/java/org/apache/solr/handler/component/QueryComponent.java
+++ b/solr/core/src/java/org/apache/solr/handler/component/QueryComponent.java
@@ -337,7 +337,7 @@ public class QueryComponent extends SearchComponent
CursorMarkParams.CURSOR_MARK_PARAM + " and " + CommonParams.TIME_ALLOWED);
}
- QueryCommand cmd = rb.getQueryCommand();
+ QueryCommand cmd = rb.createQueryCommand();
cmd.setTimeAllowed(timeAllowed);
req.getContext().put(SolrIndexSearcher.STATS_SOURCE, statsCache.get(req));
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/b8224ad6/solr/core/src/java/org/apache/solr/handler/component/ResponseBuilder.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/handler/component/ResponseBuilder.java b/solr/core/src/java/org/apache/solr/handler/component/ResponseBuilder.java
index 8e9477b..22d0465 100644
--- a/solr/core/src/java/org/apache/solr/handler/component/ResponseBuilder.java
+++ b/solr/core/src/java/org/apache/solr/handler/component/ResponseBuilder.java
@@ -421,11 +421,16 @@ public class ResponseBuilder
}
}
+ @Deprecated
+ public QueryCommand getQueryCommand() {
+ return createQueryCommand();
+ }
+
/**
* Creates a SolrIndexSearcher.QueryCommand from this
* ResponseBuilder. TimeAllowed is left unset.
*/
- public QueryCommand getQueryCommand() {
+ public QueryCommand createQueryCommand() {
QueryCommand cmd = new QueryCommand();
cmd.setQuery(wrap(getQuery()))
.setFilterList(getFilters())
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/b8224ad6/solr/core/src/java/org/apache/solr/request/SimpleFacets.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/request/SimpleFacets.java b/solr/core/src/java/org/apache/solr/request/SimpleFacets.java
index cc5559d..0b5a082 100644
--- a/solr/core/src/java/org/apache/solr/request/SimpleFacets.java
+++ b/solr/core/src/java/org/apache/solr/request/SimpleFacets.java
@@ -259,7 +259,7 @@ public class SimpleFacets {
// get the new base docset for this facet
DocSet base = searcher.getDocSet(qlist);
if (rb.grouping() && rb.getGroupingSpec().isTruncateGroups()) {
- Grouping grouping = new Grouping(searcher, null, rb.getQueryCommand(), false, 0, false);
+ Grouping grouping = new Grouping(searcher, null, rb.createQueryCommand(), false, 0, false);
grouping.setWithinGroupSort(rb.getGroupingSpec().getSortWithinGroup());
if (rb.getGroupingSpec().getFields().length > 0) {
grouping.addFieldCommand(rb.getGroupingSpec().getFields()[0], req);