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 2016/05/19 09:35:29 UTC
[29/50] lucene-solr:jira/solr-9045: SOLR-9118: HashQParserPlugin
should trim partition keys
SOLR-9118: HashQParserPlugin should trim partition keys
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/f8d10127
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/f8d10127
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/f8d10127
Branch: refs/heads/jira/solr-9045
Commit: f8d1012717620b4ed019fcf1e19e4f335fcbbe93
Parents: be51726
Author: jbernste <jb...@apache.org>
Authored: Tue May 17 14:10:34 2016 -0400
Committer: jbernste <jb...@apache.org>
Committed: Tue May 17 14:10:54 2016 -0400
----------------------------------------------------------------------
solr/core/src/java/org/apache/solr/search/HashQParserPlugin.java | 1 +
.../src/test/org/apache/solr/search/TestHashQParserPlugin.java | 4 ++--
2 files changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/f8d10127/solr/core/src/java/org/apache/solr/search/HashQParserPlugin.java
----------------------------------------------------------------------
diff --git a/solr/core/src/java/org/apache/solr/search/HashQParserPlugin.java b/solr/core/src/java/org/apache/solr/search/HashQParserPlugin.java
index e15ce1c..a95f41e 100644
--- a/solr/core/src/java/org/apache/solr/search/HashQParserPlugin.java
+++ b/solr/core/src/java/org/apache/solr/search/HashQParserPlugin.java
@@ -69,6 +69,7 @@ public class HashQParserPlugin extends QParserPlugin {
int workers = localParams.getInt("workers");
int worker = localParams.getInt("worker");
String keys = params.get("partitionKeys");
+ keys = keys.replace(" ", "");
return new HashQuery(keys, workers, worker);
}
}
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/f8d10127/solr/core/src/test/org/apache/solr/search/TestHashQParserPlugin.java
----------------------------------------------------------------------
diff --git a/solr/core/src/test/org/apache/solr/search/TestHashQParserPlugin.java b/solr/core/src/test/org/apache/solr/search/TestHashQParserPlugin.java
index 36e4fb1..8431588 100644
--- a/solr/core/src/test/org/apache/solr/search/TestHashQParserPlugin.java
+++ b/solr/core/src/test/org/apache/solr/search/TestHashQParserPlugin.java
@@ -199,7 +199,7 @@ public class TestHashQParserPlugin extends SolrTestCaseJ4 {
params = new ModifiableSolrParams();
params.add("q", "*:*");
params.add("fq", "{!hash worker=0 workers=2 cost="+getCost(random)+"}");
- params.add("partitionKeys", "a_s,a_i,a_l");
+ params.add("partitionKeys", "a_s, a_i, a_l");
params.add("rows","50");
set1 = new HashSet();
response = h.query(req(params));
@@ -217,7 +217,7 @@ public class TestHashQParserPlugin extends SolrTestCaseJ4 {
params = new ModifiableSolrParams();
params.add("q", "*:*");
params.add("fq", "{!hash worker=1 workers=2 cost="+getCost(random)+"}");
- params.add("partitionKeys", "a_s,a_i,a_l");
+ params.add("partitionKeys", "a_s, a_i, a_l");
params.add("rows","50");
set2 = new HashSet();
response = h.query(req(params));