Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 1fff861

Browse files
committed
fix NLPchina#678 range aggregation how to spec alias?
1 parent e377e5a commit 1fff861

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
<modelVersion>4.0.0</modelVersion>
44
<groupId>org.nlpcn</groupId>
55
<artifactId>elasticsearch-sql</artifactId>
6-
<version>6.2.4.5</version>
6+
<version>6.2.4.6</version>
77
<packaging>jar</packaging>
88
<description>Query elasticsearch using SQL</description>
99
<name>elasticsearch-sql</name>

src/main/java/org/nlpcn/es4sql/query/maker/AggMaker.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
import java.math.BigDecimal;
55
import java.time.ZoneOffset;
66
import java.util.*;
7+
import java.util.stream.Collectors;
78

89
import org.elasticsearch.common.xcontent.NamedXContentRegistry;
910
import org.elasticsearch.common.xcontent.XContentParser;
@@ -570,7 +571,8 @@ private HistogramAggregationBuilder histogram(MethodField field) throws SqlParse
570571
*/
571572
private RangeAggregationBuilder rangeBuilder(MethodField field) {
572573

573-
LinkedList<KVValue> params = new LinkedList<>(field.getParams());
574+
// ignore alias param
575+
LinkedList<KVValue> params = field.getParams().stream().filter(kv -> !"alias".equals(kv.key)).collect(Collectors.toCollection(LinkedList::new));
574576

575577
String fieldName = params.poll().toString();
576578

0 commit comments

Comments
 (0)