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

feat(inverted_index.search): add index applier #2

Closed
wants to merge 16 commits into from

Conversation

zhongzc
Copy link
Owner

@zhongzc zhongzc commented Dec 1, 2023

I hereby agree to the terms of the GreptimeDB CLA

What's changed and what's your intention?

PLEASE DO NOT LEAVE THIS EMPTY !!!

Please explain IN DETAIL what the changes are in this PR and why they are needed:

  • Summarize your change (mandatory)
  • How does this PR work? Need a brief introduction for the changed logic (optional)
  • Describe clearly one logical change and avoid lazy messages (optional)
  • Describe any limitations of the current code (optional)

Checklist

  • I have written the necessary rustdoc comments.
  • I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

zhongzc and others added 4 commits December 4, 2023 09:48
…2854)

* enhance empty_metric

Signed-off-by: Ruihang Xia <[email protected]>

* implementation

Signed-off-by: Ruihang Xia <[email protected]>

* fix lhs & rhs

Signed-off-by: Ruihang Xia <[email protected]>

* fix clippy

Signed-off-by: Ruihang Xia <[email protected]>

* fix typo, update sqlness

Signed-off-by: Ruihang Xia <[email protected]>

* remove deadcode

Signed-off-by: Ruihang Xia <[email protected]>

* add cast to bool modifier

Signed-off-by: Ruihang Xia <[email protected]>

* update sqlness result

Signed-off-by: Ruihang Xia <[email protected]>

---------

Signed-off-by: Ruihang Xia <[email protected]>
waynexia and others added 7 commits December 4, 2023 12:34
Signed-off-by: Zhenchi <[email protected]>
* feat(inverted_index.search): add fst applier

Signed-off-by: Zhenchi <[email protected]>

* fix: typos

Signed-off-by: Zhenchi <[email protected]>

* feat(inverted_index.search): add fst values mapper

Signed-off-by: Zhenchi <[email protected]>

* chore: remove meta check

Signed-off-by: Zhenchi <[email protected]>

* fix: fmt & clippy

Signed-off-by: Zhenchi <[email protected]>

* refactor: one expect for test

Signed-off-by: Zhenchi <[email protected]>

* chore: match error in test

Signed-off-by: Zhenchi <[email protected]>

* fix: fmt

Signed-off-by: Zhenchi <[email protected]>

* refactor: add helper function to construct fst value

Signed-off-by: Zhenchi <[email protected]>

* refactor: bytemuck to extract offset and size

Signed-off-by: Zhenchi <[email protected]>

* fix: toml format

Signed-off-by: Zhenchi <[email protected]>

---------

Signed-off-by: Zhenchi <[email protected]>
Signed-off-by: Zhenchi <[email protected]>
@zhongzc zhongzc closed this Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants