diff --git a/docs/en/changes/changes.md b/docs/en/changes/changes.md index 187d736e8779..dadf3758208a 100644 --- a/docs/en/changes/changes.md +++ b/docs/en/changes/changes.md @@ -22,6 +22,7 @@ * Fix BanyanDB time range overflow in profile thread snapshot query. * `BrowserErrorLog`, OAP Server generated UUID to replace the original client side ID, because Browser scripts can't guarantee generated IDs are globally unique. * MQE: fix multiple labeled metric query and ensure no results are returned if no label value combinations match. +* Fix `BrowserErrorLog` BanyanDB storage query order. #### UI * Fix the missing icon in new native trace view. diff --git a/oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/storage/query/IMetricsQueryDAO.java b/oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/storage/query/IMetricsQueryDAO.java index acb1ffa87ee2..9cf56f723a4b 100644 --- a/oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/storage/query/IMetricsQueryDAO.java +++ b/oap-server/server-core/src/main/java/org/apache/skywalking/oap/server/core/storage/query/IMetricsQueryDAO.java @@ -193,6 +193,7 @@ public static List composeLabelConditions(final List queryLabe keySets.add(keySet); } else { // If any query label has no matches, clear all keySets so that no results are returned keySets.clear(); + break; } } //intersection labels diff --git a/oap-server/server-storage-plugin/storage-banyandb-plugin/src/main/java/org/apache/skywalking/oap/server/storage/plugin/banyandb/stream/BanyanDBBrowserLogQueryDAO.java b/oap-server/server-storage-plugin/storage-banyandb-plugin/src/main/java/org/apache/skywalking/oap/server/storage/plugin/banyandb/stream/BanyanDBBrowserLogQueryDAO.java index 3e1c5838d541..e2ccda89562d 100644 --- a/oap-server/server-storage-plugin/storage-banyandb-plugin/src/main/java/org/apache/skywalking/oap/server/storage/plugin/banyandb/stream/BanyanDBBrowserLogQueryDAO.java +++ b/oap-server/server-storage-plugin/storage-banyandb-plugin/src/main/java/org/apache/skywalking/oap/server/storage/plugin/banyandb/stream/BanyanDBBrowserLogQueryDAO.java @@ -19,6 +19,7 @@ package org.apache.skywalking.oap.server.storage.plugin.banyandb.stream; import com.google.common.collect.ImmutableSet; +import org.apache.skywalking.library.banyandb.v1.client.AbstractQuery; import org.apache.skywalking.library.banyandb.v1.client.RowEntity; import org.apache.skywalking.library.banyandb.v1.client.StreamQuery; import org.apache.skywalking.library.banyandb.v1.client.StreamQueryResponse; @@ -70,7 +71,8 @@ public void apply(StreamQuery query) { if (Objects.nonNull(category)) { query.and(eq(BrowserErrorLogRecord.ERROR_CATEGORY, category.getValue())); } - + query.setOrderBy( + new StreamQuery.OrderBy(AbstractQuery.Sort.DESC)); query.setOffset(from); query.setLimit(limit); }