[improve](ip-function)improve is_ip_address_in_range for inverted index speed #41768
Conversation
|
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
|
run buildall |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
TeamCity be ut coverage result: |
regression-test/suites/inverted_index_p0/test_ip_cidr_search_with_inverted_index.groovy
Show resolved
Hide resolved
|
PR approved by anyone and no changes requested. |
|
run buildall |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| String get_name() const override { return name; } | ||
|
|
||
| size_t get_number_of_arguments() const override { return 2; } | ||
| Status evaluate_inverted_index( |
There was a problem hiding this comment.
warning: function 'evaluate_inverted_index' exceeds recommended size/complexity thresholds [readability-function-size]
Status evaluate_inverted_index(
^Additional context
be/src/vec/functions/function_ip.h:641: 90 lines including whitespace and comments (threshold 80)
Status evaluate_inverted_index(
^
|
TeamCity be ut coverage result: |
|
PR approved by at least one committer and no changes requested. |
github-actions
bot
added
the
approved
…ex speed (apache#41768) ## Proposed changes speed up is_ip_address_in_range with inverted index
…ex speed (apache#41768) speed up is_ip_address_in_range with inverted index
…ex speed (apache#41768) speed up is_ip_address_in_range with inverted index
6 participants
Proposed changes
speed up is_ip_address_in_range with inverted index
Issue Number: close #xxx