Skip to content

Commit f965132

Browse files
committed
Plan correct encoder for exponential histograms
1 parent e4fa25a commit f965132

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/planner/LocalExecutionPlanner.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -487,10 +487,9 @@ private PhysicalOperation planTopN(TopNExec topNExec, LocalExecutionPlannerConte
487487
case BOOLEAN, NULL, BYTE, SHORT, INTEGER, LONG, DOUBLE, FLOAT, HALF_FLOAT, DATETIME, DATE_NANOS, DATE_PERIOD, TIME_DURATION,
488488
OBJECT, SCALED_FLOAT, UNSIGNED_LONG, TSID_DATA_TYPE -> TopNEncoder.DEFAULT_SORTABLE;
489489
case GEO_POINT, CARTESIAN_POINT, GEO_SHAPE, CARTESIAN_SHAPE, COUNTER_LONG, COUNTER_INTEGER, COUNTER_DOUBLE, SOURCE,
490-
AGGREGATE_METRIC_DOUBLE, DENSE_VECTOR, GEOHASH, GEOTILE, GEOHEX -> TopNEncoder.DEFAULT_UNSORTABLE;
490+
AGGREGATE_METRIC_DOUBLE, DENSE_VECTOR, GEOHASH, GEOTILE, GEOHEX, EXPONENTIAL_HISTOGRAM -> TopNEncoder.DEFAULT_UNSORTABLE;
491491
// unsupported fields are encoded as BytesRef, we'll use the same encoder; all values should be null at this point
492-
// TODO(b/133393) add exponential histogram support
493-
case PARTIAL_AGG, EXPONENTIAL_HISTOGRAM, UNSUPPORTED -> TopNEncoder.UNSUPPORTED;
492+
case PARTIAL_AGG, UNSUPPORTED -> TopNEncoder.UNSUPPORTED;
494493
};
495494
}
496495
List<TopNOperator.SortOrder> orders = topNExec.order().stream().map(order -> {

0 commit comments

Comments
 (0)