-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[feature](index change)Support light index change for ngram bf index #48461
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 |
|
run compile |
679e17b to
d54f726
Compare
|
run buildall |
2 similar comments
|
run buildall |
|
run buildall |
d54f726 to
6091a62
Compare
|
run buildall |
TPC-H: Total hot run time: 31476 ms |
TPC-DS: Total hot run time: 191500 ms |
ClickBench: Total hot run time: 30.62 s |
6091a62 to
7d08604
Compare
|
run buildall |
TPC-H: Total hot run time: 32307 ms |
TPC-DS: Total hot run time: 185622 ms |
ClickBench: Total hot run time: 30.68 s |
7d08604 to
4ede4ef
Compare
|
run buildall |
TPC-H: Total hot run time: 32126 ms |
TPC-DS: Total hot run time: 192143 ms |
ClickBench: Total hot run time: 31.1 s |
4ede4ef to
f25084a
Compare
|
run buildall |
f25084a to
8723c95
Compare
|
run buildall |
TPC-H: Total hot run time: 32896 ms |
TPC-DS: Total hot run time: 185891 ms |
ClickBench: Total hot run time: 31.13 s |
8723c95 to
6a7dbbf
Compare
283baf0 to
0211cdd
Compare
|
run buildall |
TPC-H: Total hot run time: 33875 ms |
TPC-DS: Total hot run time: 192363 ms |
ClickBench: Total hot run time: 28.98 s |
airborne12
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. |
zzzxl1993
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
…pache#48461) ### What problem does this PR solve? Currently, NGram bloom filter index only supports directly schema change, and users need to build indexes incrementally when using it. The design goal is that ngrambf supports light_index_change, including local and cloud mode, which can incrementally add indexes or build indexes on stock data. Inverted indexes are currently only supported in local mode for light_schema_change, cloud mode is still a directly SC, this time it does not involve inverted indexes, and its functionality remains unchanged. After the completion of the function, the NGram BF index construction can be used in the following way, following the existing syntax, does not involve changes or additions. ```sql # add new index alter table t1 add index idx_ngram_k2 (`k2`) using ngram_bf properties("bf_size" = "1024", "gram_size" = "3"); create index idx_ngram_k2 (`k2`) on t1 using ngram_bf properties("bf_size" = "1024", "gram_size" = "3"); # build index on stock data build index idx_ngram_k2 on t1; # show build ngram index show alter table column; # cancel build index cancel build index on t1; ``` **NOTE:** Currently, building an index by partition is not supported. If you want to build an index for stock data, you need to build it for all data, including new data written after the Add index has been added. Build index by partition will be supported in next stage. ### Release note Support light index change for NGram bf index
…pache#48461) Currently, NGram bloom filter index only supports directly schema change, and users need to build indexes incrementally when using it. The design goal is that ngrambf supports light_index_change, including local and cloud mode, which can incrementally add indexes or build indexes on stock data. Inverted indexes are currently only supported in local mode for light_schema_change, cloud mode is still a directly SC, this time it does not involve inverted indexes, and its functionality remains unchanged. After the completion of the function, the NGram BF index construction can be used in the following way, following the existing syntax, does not involve changes or additions. ```sql alter table t1 add index idx_ngram_k2 (`k2`) using ngram_bf properties("bf_size" = "1024", "gram_size" = "3"); create index idx_ngram_k2 (`k2`) on t1 using ngram_bf properties("bf_size" = "1024", "gram_size" = "3"); build index idx_ngram_k2 on t1; show alter table column; cancel build index on t1; ``` **NOTE:** Currently, building an index by partition is not supported. If you want to build an index for stock data, you need to build it for all data, including new data written after the Add index has been added. Build index by partition will be supported in next stage. Support light index change for NGram bf index
…pache#48461) Currently, NGram bloom filter index only supports directly schema change, and users need to build indexes incrementally when using it. The design goal is that ngrambf supports light_index_change, including local and cloud mode, which can incrementally add indexes or build indexes on stock data. Inverted indexes are currently only supported in local mode for light_schema_change, cloud mode is still a directly SC, this time it does not involve inverted indexes, and its functionality remains unchanged. After the completion of the function, the NGram BF index construction can be used in the following way, following the existing syntax, does not involve changes or additions. ```sql alter table t1 add index idx_ngram_k2 (`k2`) using ngram_bf properties("bf_size" = "1024", "gram_size" = "3"); create index idx_ngram_k2 (`k2`) on t1 using ngram_bf properties("bf_size" = "1024", "gram_size" = "3"); build index idx_ngram_k2 on t1; show alter table column; cancel build index on t1; ``` **NOTE:** Currently, building an index by partition is not supported. If you want to build an index for stock data, you need to build it for all data, including new data written after the Add index has been added. Build index by partition will be supported in next stage. Support light index change for NGram bf index
…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?
Currently, NGram bloom filter index only supports directly schema change, and users need to build indexes incrementally when using it.
The design goal is that ngrambf supports light_index_change, including local and cloud mode, which can incrementally add indexes or build indexes on stock data.
Inverted indexes are currently only supported in local mode for light_schema_change, cloud mode is still a directly SC, this time it does not involve inverted indexes, and its functionality remains unchanged.
After the completion of the function, the NGram BF index construction can be used in the following way, following the existing syntax, does not involve changes or additions.
NOTE: Currently, building an index by partition is not supported. If you want to build an index for stock data, you need to build it for all data, including new data written after the Add index has been added.
Build index by partition will be supported in next stage.
Release note
Support light index change for NGram bf index
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)