diff --git a/docs/en/changes/changes.md b/docs/en/changes/changes.md index cc33e408d636c3ecd16f9c9ed78e843cd0c2f3af..bf8ed4a5e820f1732aa80aa9191e859f9c42c279 100644 --- a/docs/en/changes/changes.md +++ b/docs/en/changes/changes.md @@ -64,6 +64,7 @@ * [Breaking Change] SQL Database: move `Tags list` from `Segment`,`Logs`,`Alarms` to their additional table. * [Breaking Change] Remove `total` field in Trace, Log, Event, Browser log, and alarm list query. * Support `OFF_CPU` eBPF Profiling. +* Fix SumAggregationBuilder#build should use the SumAggregation rather than MaxAggregation. #### UI diff --git a/oap-server/server-library/library-elasticsearch-client/src/main/java/org/apache/skywalking/library/elasticsearch/requests/search/aggregation/SumAggregationBuilder.java b/oap-server/server-library/library-elasticsearch-client/src/main/java/org/apache/skywalking/library/elasticsearch/requests/search/aggregation/SumAggregationBuilder.java index b7a7f888838dec64a7ab193fbb20930d22a1995d..e949fd79e7a8fbb50ad3db8dddfdc2e64745be6f 100644 --- a/oap-server/server-library/library-elasticsearch-client/src/main/java/org/apache/skywalking/library/elasticsearch/requests/search/aggregation/SumAggregationBuilder.java +++ b/oap-server/server-library/library-elasticsearch-client/src/main/java/org/apache/skywalking/library/elasticsearch/requests/search/aggregation/SumAggregationBuilder.java @@ -37,7 +37,7 @@ public final class SumAggregationBuilder implements AggregationBuilder { } @Override - public MaxAggregation build() { - return new MaxAggregation(name, field); + public SumAggregation build() { + return new SumAggregation(name, field); } }