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
docs: rfc of inverted index for SST file #2693
docs: rfc of inverted index for SST file #2693
Conversation
Signed-off-by: Zhenchi <zhongzc_arch@outlook.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Question: if we want to support other index types such as geospatial indexing, is the API design extensible?
Signed-off-by: Zhenchi <zhongzc_arch@outlook.com>
Signed-off-by: Zhenchi <zhongzc_arch@outlook.com>
Signed-off-by: Zhenchi <zhongzc_arch@outlook.com>
We have designed the Index data to be housed within Puffin. As a container, Puffin opens up opportunities for us to iterate formats and develop other types of indexes. Inverted index data with the format proposed in this RFC will be placed in Puffin as a blob, and will be assigned the type When there is a need for a geospatial index, we can design a new format in the same file and assign a new type like |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Cool, let's make it work. |
I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
rendered
Checklist
Refer to a related PR or issue link (optional)