Uh oh!
There was an error while loading. Please reload this page.
[Refactor](inverted index) refactor inverted index interface in tablet schema - #43003
Conversation
doris-robot
commented
Oct 31, 2024
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
csun5285
commented
Oct 31, 2024
run buildall |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
doris-robot
commented
Oct 31, 2024
TPC-H: Total hot run time: 41471 ms |
doris-robot
commented
Oct 31, 2024
TeamCity be ut coverage result: |
doris-robot
commented
Oct 31, 2024
TPC-DS: Total hot run time: 191990 ms |
doris-robot
commented
Oct 31, 2024
ClickBench: Total hot run time: 33.06 s |
csun5285
commented
Oct 31, 2024
run buildall |
18df067 to
79b8c37Comparecsun5285
commented
Oct 31, 2024
run buildall |
1 similar comment
csun5285
commented
Oct 31, 2024
run buildall |
doris-robot
commented
Oct 31, 2024
TeamCity be ut coverage result: |
doris-robot
commented
Nov 1, 2024
TPC-H: Total hot run time: 41336 ms |
doris-robot
commented
Nov 1, 2024
TPC-DS: Total hot run time: 195526 ms |
doris-robot
commented
Nov 1, 2024
ClickBench: Total hot run time: 31.8 s |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
70bca58 to
b6e22ecComparecsun5285
commented
Nov 4, 2024
run buildall |
doris-robot
commented
Nov 4, 2024
TeamCity be ut coverage result: |
doris-robot
commented
Nov 4, 2024
TPC-H: Total hot run time: 41409 ms |
doris-robot
commented
Nov 4, 2024
TPC-DS: Total hot run time: 195949 ms |
doris-robot
commented
Nov 4, 2024
ClickBench: Total hot run time: 33.16 s |
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
csun5285
commented
Nov 4, 2024
run buildall |
csun5285
commented
Nov 4, 2024
run buildall |
doris-robot
commented
Nov 4, 2024
TPC-H: Total hot run time: 41395 ms |
doris-robot
commented
Nov 4, 2024
TPC-DS: Total hot run time: 191115 ms |
doris-robot
commented
Nov 4, 2024
ClickBench: Total hot run time: 33.07 s |
doris-robot
commented
Nov 4, 2024
TeamCity be ut coverage result: |
airborne12
commented
Nov 5, 2024
run p1 |
airborne12
commented
Nov 5, 2024
run cloud_p1 |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
…t schema (apache#43003) ## Proposed changes refactor inverted index interface in tablet schema 1. There's a large amount of similar code ``` for (const auto& index : schema_ptr->indexes()) { if (index.index_type() == IndexType::INVERTED) { .... } } ``` Now it has been changed to ``` for (const auto& index : schema_ptr->inverted_indexes()) { .... } ``` 2. Remove functions with redundant functionality, such as `get_indexes_for_column `, `get_inverted_index_with_index_id` .... 3. Retrieve the index meta for the inverted index. Currently, there are two types: one is `inverted_index(TabletColumn)`, which checks the column type (e.g., if the column type is float, it does not support indexing and will be filtered out); the other is `inverted_index(unique_id, index_suffix)`, which retrieves the index meta for the current column without checking the column type. <!--Describe your changes.-->
…t schema (apache#43003) ## Proposed changes refactor inverted index interface in tablet schema 1. There's a large amount of similar code ``` for (const auto& index : schema_ptr->indexes()) { if (index.index_type() == IndexType::INVERTED) { .... } } ``` Now it has been changed to ``` for (const auto& index : schema_ptr->inverted_indexes()) { .... } ``` 2. Remove functions with redundant functionality, such as `get_indexes_for_column `, `get_inverted_index_with_index_id` .... 3. Retrieve the index meta for the inverted index. Currently, there are two types: one is `inverted_index(TabletColumn)`, which checks the column type (e.g., if the column type is float, it does not support indexing and will be filtered out); the other is `inverted_index(unique_id, index_suffix)`, which retrieves the index meta for the current column without checking the column type. <!--Describe your changes.-->
Proposed changes
refactor inverted index interface in tablet schema
Now it has been changed to
Remove functions with redundant functionality, such as
get_indexes_for_column,get_inverted_index_with_index_id....Retrieve the index meta for the inverted index. Currently, there are two types: one is
inverted_index(TabletColumn), which checks the column type (e.g., if the column type is float, it does not support indexing and will be filtered out); the other isinverted_index(unique_id, index_suffix), which retrieves the index meta for the current column without checking the column type.