-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[feature](index change)Support light index change for inverted index without parser #52251
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
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
bf75dae to
9cca0dd
Compare
|
run buildall |
1 similar comment
|
run buildall |
TPC-H: Total hot run time: 34178 ms |
TPC-DS: Total hot run time: 185388 ms |
|
run buildall |
TPC-H: Total hot run time: 33843 ms |
TPC-DS: Total hot run time: 185998 ms |
ClickBench: Total hot run time: 30.19 s |
|
run buildall |
TPC-H: Total hot run time: 34254 ms |
TPC-DS: Total hot run time: 186571 ms |
ClickBench: Total hot run time: 29.06 s |
|
run buildall |
1 similar comment
|
run buildall |
TPC-H: Total hot run time: 33949 ms |
TPC-DS: Total hot run time: 186650 ms |
|
run buildall |
TPC-H: Total hot run time: 33777 ms |
TPC-DS: Total hot run time: 185513 ms |
ClickBench: Total hot run time: 29.57 s |
|
run buildall |
FE UT Coverage ReportIncrement line coverage |
eldenmoon
left a comment
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
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
dataroaring
left a comment
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
…without parser (apache#52251) Problem Summary: This PR adds support for lightweight index add for inverted indexes without parser in cloud mode and extends the existing “light” mode for NGRAM Bloom Filter indexes. And also introduces a new session variable (enable_add_index_for_new_data) to control the above logic.
…without parser (apache#52251) Problem Summary: This PR adds support for lightweight index add for inverted indexes without parser in cloud mode and extends the existing “light” mode for NGRAM Bloom Filter indexes. And also introduces a new session variable (enable_add_index_for_new_data) to control the above logic.
…without parser (apache#52251) Problem Summary: This PR adds support for lightweight index add for inverted indexes without parser in cloud mode and extends the existing “light” mode for NGRAM Bloom Filter indexes. And also introduces a new session variable (enable_add_index_for_new_data) to control the above logic.
…without parser (apache#52251) Problem Summary: This PR adds support for lightweight index add for inverted indexes without parser in cloud mode and extends the existing “light” mode for NGRAM Bloom Filter indexes. And also introduces a new session variable (enable_add_index_for_new_data) to control the above logic.
…without parser (apache#52251) Problem Summary: This PR adds support for lightweight index add for inverted indexes without parser in cloud mode and extends the existing “light” mode for NGRAM Bloom Filter indexes. And also introduces a new session variable (enable_add_index_for_new_data) to control the above logic.
…x without parser and ngram bf index apache#48461 apache#52251 (apache#52894) cherry pick from apache#48461 and apache#52251 --------- Co-authored-by: qiye <jianliang5669@gmail.com>
What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
This PR adds support for lightweight index add for inverted indexes without parser in cloud mode and extends the existing “light” mode for NGRAM Bloom Filter indexes. And also introduces a new session variable (enable_add_index_for_new_data) to control the above logic.
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)