From 8b40d2269f22f3d22d6f821794f90bbd7ffaeaf1 Mon Sep 17 00:00:00 2001 From: yangyiweigege <2873824885@qq.com> Date: Sat, 21 May 2022 19:01:03 +0800 Subject: [PATCH] Fix SumAggregationBuilder#build should use the SumAggregation rather than MaxAggregation.(#9111) (#9114) Co-authored-by: xishui --- docs/en/changes/changes.md | 1 + .../requests/search/aggregation/SumAggregationBuilder.java | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/docs/en/changes/changes.md b/docs/en/changes/changes.md index cc33e408d6..bf8ed4a5e8 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 b7a7f88883..e949fd79e7 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); } } -- GitLab