You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by jb...@apache.org on 2017/06/21 14:17:55 UTC
lucene-solr:branch_6x: SOLR-10933: LetStream variables are not
evaluated in proper order
Repository: lucene-solr
Updated Branches:
refs/heads/branch_6x 9d3415bcf -> adfaf340e
SOLR-10933: LetStream variables are not evaluated in proper order
Project: http://git-wip-us.apache.org/repos/asf/lucene-solr/repo
Commit: http://git-wip-us.apache.org/repos/asf/lucene-solr/commit/adfaf340
Tree: http://git-wip-us.apache.org/repos/asf/lucene-solr/tree/adfaf340
Diff: http://git-wip-us.apache.org/repos/asf/lucene-solr/diff/adfaf340
Branch: refs/heads/branch_6x
Commit: adfaf340e027ac73a672f6916f0e9be72cd9e3d1
Parents: 9d3415b
Author: Joel Bernstein <jb...@apache.org>
Authored: Wed Jun 21 10:01:40 2017 -0400
Committer: Joel Bernstein <jb...@apache.org>
Committed: Wed Jun 21 10:13:58 2017 -0400
----------------------------------------------------------------------
.../java/org/apache/solr/client/solrj/io/stream/LetStream.java | 4 ++--
.../apache/solr/client/solrj/io/stream/StreamExpressionTest.java | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/adfaf340/solr/solrj/src/java/org/apache/solr/client/solrj/io/stream/LetStream.java
----------------------------------------------------------------------
diff --git a/solr/solrj/src/java/org/apache/solr/client/solrj/io/stream/LetStream.java b/solr/solrj/src/java/org/apache/solr/client/solrj/io/stream/LetStream.java
index 8c9f02b..6dd73f5 100644
--- a/solr/solrj/src/java/org/apache/solr/client/solrj/io/stream/LetStream.java
+++ b/solr/solrj/src/java/org/apache/solr/client/solrj/io/stream/LetStream.java
@@ -18,7 +18,7 @@ package org.apache.solr.client.solrj.io.stream;
import java.io.IOException;
import java.util.ArrayList;
-import java.util.HashMap;
+import java.util.LinkedHashMap;
import java.util.List;
import java.util.Locale;
import java.util.Map;
@@ -41,7 +41,7 @@ public class LetStream extends TupleStream implements Expressible {
private static final long serialVersionUID = 1;
private TupleStream stream;
private StreamContext streamContext;
- private Map letParams = new HashMap();
+ private Map letParams = new LinkedHashMap();
public LetStream(StreamExpression expression, StreamFactory factory) throws IOException {
List<StreamExpression> streamExpressions = factory.getExpressionOperandsRepresentingTypes(expression, Expressible.class, TupleStream.class);
http://git-wip-us.apache.org/repos/asf/lucene-solr/blob/adfaf340/solr/solrj/src/test/org/apache/solr/client/solrj/io/stream/StreamExpressionTest.java
----------------------------------------------------------------------
diff --git a/solr/solrj/src/test/org/apache/solr/client/solrj/io/stream/StreamExpressionTest.java b/solr/solrj/src/test/org/apache/solr/client/solrj/io/stream/StreamExpressionTest.java
index dd8e7ff..854a5c0 100644
--- a/solr/solrj/src/test/org/apache/solr/client/solrj/io/stream/StreamExpressionTest.java
+++ b/solr/solrj/src/test/org/apache/solr/client/solrj/io/stream/StreamExpressionTest.java
@@ -6467,7 +6467,7 @@ public class StreamExpressionTest extends SolrCloudTestCase {
updateRequest.commit(cluster.getSolrClient(), COLLECTIONORALIAS);
String expr = "search("+COLLECTIONORALIAS+", q=\"*:*\", fl=\"id,test_t, test_i\", sort=\"id desc\")";
- String cat = "let(a ="+expr+", b = add(1,3), c=col(a, test_i), tuple(test = add(1,1), test1=b, results=a, test2=add(c)))";
+ String cat = "let(d ="+expr+", b = add(1,3), c=col(d, test_i), tuple(test = add(1,1), test1=b, results=d, test2=add(c)))";
ModifiableSolrParams paramsLoc = new ModifiableSolrParams();
paramsLoc.set("expr", cat);
paramsLoc.set("qt", "/stream");