Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BanyanDB-Server] Improve sorting and filtering performance of Stream #12160

Closed
2 of 3 tasks
hanahmily opened this issue Apr 24, 2024 · 3 comments · Fixed by apache/skywalking-banyandb#440
Closed
2 of 3 tasks
Assignees
Labels
database BanyanDB - SkyWalking native database enhancement Enhancement on performance or codes

Comments

@hanahmily
Copy link
Contributor

Search before asking

  • I had searched in the issues and found no similar feature requirement.

Description

The current filtering and sorting operations scan through all parts to locate a single record in a loop. As the number of parts increases, this method can cause delays and slow down the scanning process.

To resolve this issue, the operations should refer to the standard query, which leverages the tsTableIterator to scan parts only once.

Use case

No response

Related issues

No response

Are you willing to submit a pull request to implement this on your own?

  • Yes I am willing to submit a pull request on my own!

Code of Conduct

@hanahmily hanahmily added enhancement Enhancement on performance or codes database BanyanDB - SkyWalking native database labels Apr 24, 2024
@wu-sheng
Copy link
Member

Is this in 0.6 or next 0.7?

@hanahmily
Copy link
Contributor Author

Is this in 0.6 or next 0.7?

It will be in the next one, so I didn't tag it as 0.6.

@wu-sheng wu-sheng added this to the BanyanDB - 0.7.0 milestone Apr 24, 2024
@ButterBright
Copy link
Member

I'd like to contribute, please assign it to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
database BanyanDB - SkyWalking native database enhancement Enhancement on performance or codes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants