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

Optimize query performance of series index #491

Merged
merged 21 commits into from
Aug 3, 2024
Merged

Conversation

ButterBright
Copy link
Member

@ButterBright ButterBright commented Jul 22, 2024

@ButterBright ButterBright added the enhancement New feature or request label Jul 22, 2024
@ButterBright ButterBright added this to the 0.7.0 milestone Jul 22, 2024
@wu-sheng
Copy link
Member

Is there benchmark or some reports to indicate how much this will improve?

@ButterBright
Copy link
Member Author

Not yet. Let me add one.

Copy link
Contributor

@hanahmily hanahmily left a comment

Choose a reason for hiding this comment

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

As we discussed before, the bluge's structs lack the String needed to build the tracing tree. Furthermore, the bulge, as a third-party library, should not spread out the inverted package. Considering these two reasons, you would have to design a series of structs to encapsulate bluge queries.

pkg/query/logical/index_filter.go Outdated Show resolved Hide resolved
pkg/query/logical/index_filter.go Outdated Show resolved Hide resolved
pkg/query/logical/index_filter.go Outdated Show resolved Hide resolved
pkg/index/inverted/inverted.go Show resolved Hide resolved
banyand/internal/storage/index.go Show resolved Hide resolved
banyand/internal/storage/storage.go Outdated Show resolved Hide resolved
pkg/index/index.go Outdated Show resolved Hide resolved
pkg/index/inverted/index_filter.go Outdated Show resolved Hide resolved
pkg/index/inverted/index_filter.go Outdated Show resolved Hide resolved
pkg/index/inverted/index_filter.go Outdated Show resolved Hide resolved
pkg/index/inverted/index_filter.go Outdated Show resolved Hide resolved
pkg/index/inverted/inverted.go Outdated Show resolved Hide resolved
@wu-sheng wu-sheng merged commit 198129c into apache:main Aug 3, 2024
9 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BanyanDB-Server] Optimizing Series Index Querying Performance
3 participants