Skip to content

Commit 97e89d9

Browse files
committed
Fix order
Signed-off-by: Lantao Jin <ltjin@amazon.com>
1 parent 90449b6 commit 97e89d9

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

opensearch/src/main/java/org/opensearch/sql/opensearch/storage/script/aggregation/dsl/BucketAggregationBuilder.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212

1313
import java.util.List;
1414
import java.util.stream.Collectors;
15+
import org.opensearch.search.aggregations.BucketOrder;
1516
import org.opensearch.search.aggregations.bucket.histogram.DateHistogramAggregationBuilder;
1617
import org.opensearch.search.aggregations.bucket.histogram.DateHistogramInterval;
1718
import org.opensearch.search.aggregations.bucket.histogram.HistogramAggregationBuilder;
@@ -47,6 +48,7 @@ public ValuesSourceAggregationBuilder<?> build(NamedExpression expr) {
4748
} else {
4849
TermsAggregationBuilder sourceBuilder = new TermsAggregationBuilder(expr.getName());
4950
sourceBuilder.size(AGGREGATION_BUCKET_SIZE);
51+
sourceBuilder.order(BucketOrder.key(true));
5052
// Time types values are converted to LONG in ExpressionAggregationScript::execute
5153
if ((expr.getDelegated().type() instanceof OpenSearchDateType
5254
&& List.of(TIMESTAMP, TIME, DATE)

0 commit comments

Comments
 (0)