Skip to content

Conversation

@wankai123
Copy link
Member

  • If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
  • Update the CHANGES log.

@wankai123 wankai123 added this to the 10.4.0 milestone Jan 15, 2026
@wankai123 wankai123 requested review from Copilot and wu-sheng January 15, 2026 07:53
@wankai123 wankai123 added feature New feature database BanyanDB - SkyWalking native database labels Jan 15, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds Order By support to the BanyanDB Property query client. The implementation allows property queries to be sorted by tag name in ascending or descending order.

Changes:

  • Added OrderBy inner class to PropertyQuery with support for sorting by tag name
  • Integrated OrderBy into the property query build process
  • Updated changelog to document the new feature

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
oap-server/server-library/library-banyandb-client/src/main/java/org/apache/skywalking/library/banyandb/v1/client/PropertyQuery.java Added OrderBy nested class and integrated it into the query builder to support sorting property queries by tag name
docs/en/changes/changes.md Added changelog entry documenting the new Order By support for property queries

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@wankai123 wankai123 merged commit 95182a7 into apache:master Jan 15, 2026
354 of 357 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

database BanyanDB - SkyWalking native database feature New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants