From 5be493b56a1cc37018d6de31fbe87f9ffc98b00b Mon Sep 17 00:00:00 2001 From: currenjin Date: Fri, 3 Apr 2026 08:47:35 +0900 Subject: [PATCH] Remove GroupBy.field_name from BanyanDB MeasureQuery request building --- docs/en/changes/changes.md | 1 + .../skywalking/library/banyandb/v1/client/MeasureQuery.java | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/en/changes/changes.md b/docs/en/changes/changes.md index 4d6478e11040..5d95f03af0fe 100644 --- a/docs/en/changes/changes.md +++ b/docs/en/changes/changes.md @@ -6,6 +6,7 @@ * Add Zipkin Virtual GenAI e2e test. Use `zipkin_json` exporter to avoid protobuf dependency conflict between `opentelemetry-exporter-zipkin-proto-http` (protobuf~=3.12) and `opentelemetry-proto` (protobuf>=5.0). * Fix missing `taskId` filter and incorrect `IN` clause parameter binding in `JDBCJFRDataQueryDAO` and `JDBCPprofDataQueryDAO`. +* Remove deprecated `GroupBy.field_name` from BanyanDB `MeasureQuery` request building (Phase 1 of staged removal across repos). #### UI diff --git a/oap-server/server-library/library-banyandb-client/src/main/java/org/apache/skywalking/library/banyandb/v1/client/MeasureQuery.java b/oap-server/server-library/library-banyandb-client/src/main/java/org/apache/skywalking/library/banyandb/v1/client/MeasureQuery.java index 5b0251c34996..4dde52699a67 100644 --- a/oap-server/server-library/library-banyandb-client/src/main/java/org/apache/skywalking/library/banyandb/v1/client/MeasureQuery.java +++ b/oap-server/server-library/library-banyandb-client/src/main/java/org/apache/skywalking/library/banyandb/v1/client/MeasureQuery.java @@ -182,7 +182,6 @@ BanyandbMeasure.QueryRequest build() throws BanyanDBException { throw new IllegalArgumentException("field name cannot be null or empty if aggregation is specified"); } } else { - groupByBuilder.setFieldName(this.aggregation.fieldName); BanyandbMeasure.QueryRequest.Aggregation aggr = BanyandbMeasure.QueryRequest.Aggregation.newBuilder() .setFunction(this.aggregation.aggregationType.function) .setFieldName(this.aggregation.fieldName)