This is an automated email from the ASF dual-hosted git repository.
wankai pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/skywalking.git
The following commit(s) were added to refs/heads/master by this push:
new 176d190969 Fix `BrowserErrorLog` BanyanDB storage query order. (#13666)
176d190969 is described below
commit 176d19096972d0f89c1c25f02a51384ad5cc83c1
Author: Wan Kai <[email protected]>
AuthorDate: Wed Jan 14 16:35:26 2026 +0800
Fix `BrowserErrorLog` BanyanDB storage query order. (#13666)
---
docs/en/changes/changes.md | 1 +
.../skywalking/oap/server/core/storage/query/IMetricsQueryDAO.java | 1 +
.../storage/plugin/banyandb/stream/BanyanDBBrowserLogQueryDAO.java | 4 +++-
3 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/docs/en/changes/changes.md b/docs/en/changes/changes.md
index 187d736e87..dadf375820 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 acb1ffa87e..9cf56f723a 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 interface IMetricsQueryDAO extends DAO {
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 3e1c5838d5..e2ccda8956 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 class BanyanDBBrowserLogQueryDAO extends
AbstractBanyanDBDAO implements I
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);
}