Skip to content

Commit 53cde65

Browse files
authored
Fix compile issue, add geo module as dependency (#808)
* Fix compile issue, add geo module as dependency Signed-off-by: penghuo <[email protected]>
1 parent 10fe75f commit 53cde65

File tree

3 files changed

+8
-5
lines changed

3 files changed

+8
-5
lines changed

legacy/build.gradle

+2
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,8 @@ dependencies {
9292
implementation group: 'org.json', name: 'json', version:'20180813'
9393
implementation group: 'org.apache.commons', name: 'commons-lang3', version: '3.10'
9494
implementation group: 'org.opensearch', name: 'opensearch', version: "${opensearch_version}"
95+
// add geo module as dependency. https://github.com/opensearch-project/OpenSearch/pull/4180/.
96+
implementation group: 'org.opensearch.plugin', name: 'geo', version: "${opensearch_version}"
9597
api project(':sql')
9698
api project(':common')
9799
api project(':opensearch')

legacy/src/main/java/org/opensearch/sql/legacy/executor/csv/CSVResultsExtractor.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
import org.opensearch.search.aggregations.bucket.MultiBucketsAggregation;
2222
import org.opensearch.search.aggregations.bucket.SingleBucketAggregation;
2323
import org.opensearch.search.aggregations.metrics.ExtendedStats;
24-
import org.opensearch.search.aggregations.metrics.GeoBounds;
24+
import org.opensearch.geo.search.aggregations.metrics.GeoBounds;
2525
import org.opensearch.search.aggregations.metrics.NumericMetricsAggregation;
2626
import org.opensearch.search.aggregations.metrics.Percentile;
2727
import org.opensearch.search.aggregations.metrics.Percentiles;

legacy/src/main/java/org/opensearch/sql/legacy/query/maker/AggMaker.java

+5-4
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
import org.opensearch.common.xcontent.XContentParser;
2626
import org.opensearch.common.xcontent.json.JsonXContent;
2727
import org.opensearch.common.xcontent.json.JsonXContentParser;
28+
import org.opensearch.geo.search.aggregations.bucket.geogrid.GeoHashGridAggregationBuilder;
2829
import org.opensearch.join.aggregations.JoinAggregationBuilders;
2930
import org.opensearch.script.Script;
3031
import org.opensearch.script.ScriptType;
@@ -34,7 +35,7 @@
3435
import org.opensearch.search.aggregations.BucketOrder;
3536
import org.opensearch.search.aggregations.InternalOrder;
3637
import org.opensearch.search.aggregations.bucket.filter.FilterAggregationBuilder;
37-
import org.opensearch.search.aggregations.bucket.geogrid.GeoGridAggregationBuilder;
38+
import org.opensearch.geo.search.aggregations.bucket.geogrid.GeoGridAggregationBuilder;
3839
import org.opensearch.search.aggregations.bucket.histogram.DateHistogramAggregationBuilder;
3940
import org.opensearch.search.aggregations.bucket.histogram.DateHistogramInterval;
4041
import org.opensearch.search.aggregations.bucket.histogram.HistogramAggregationBuilder;
@@ -44,7 +45,7 @@
4445
import org.opensearch.search.aggregations.bucket.range.RangeAggregationBuilder;
4546
import org.opensearch.search.aggregations.bucket.terms.IncludeExclude;
4647
import org.opensearch.search.aggregations.bucket.terms.TermsAggregationBuilder;
47-
import org.opensearch.search.aggregations.metrics.GeoBoundsAggregationBuilder;
48+
import org.opensearch.geo.search.aggregations.metrics.GeoBoundsAggregationBuilder;
4849
import org.opensearch.search.aggregations.metrics.PercentilesAggregationBuilder;
4950
import org.opensearch.search.aggregations.metrics.ScriptedMetricAggregationBuilder;
5051
import org.opensearch.search.aggregations.metrics.TopHitsAggregationBuilder;
@@ -285,7 +286,7 @@ private AggregationBuilder makeRangeGroup(MethodField field) throws SqlParseExce
285286

286287
private AggregationBuilder geoBounds(MethodField field) throws SqlParseException {
287288
String aggName = gettAggNameFromParamsOrAlias(field);
288-
GeoBoundsAggregationBuilder boundsBuilder = AggregationBuilders.geoBounds(aggName);
289+
GeoBoundsAggregationBuilder boundsBuilder = new GeoBoundsAggregationBuilder(aggName);
289290
String value;
290291
for (KVValue kv : field.getParams()) {
291292
value = kv.value.toString();
@@ -472,7 +473,7 @@ private AbstractAggregationBuilder scriptedMetric(MethodField field) throws SqlP
472473

473474
private AggregationBuilder geohashGrid(MethodField field) throws SqlParseException {
474475
String aggName = gettAggNameFromParamsOrAlias(field);
475-
GeoGridAggregationBuilder geoHashGrid = AggregationBuilders.geohashGrid(aggName);
476+
GeoGridAggregationBuilder geoHashGrid = new GeoHashGridAggregationBuilder(aggName);
476477
String value;
477478
for (KVValue kv : field.getParams()) {
478479
value = kv.value.toString();

0 commit comments

Comments
 (0)