-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[opt](ann index) Support cast expression as rhs of approximate top n #55458
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:
|
3229885 to
46b15a6
Compare
Possible file(s) that should be tracked in LFS detected: 🚨The following file(s) exceeds the file size limit:
Consider using |
46b15a6 to
ad66d75
Compare
Possible file(s) that should be tracked in LFS detected: 🚨The following file(s) exceeds the file size limit:
Consider using |
ad66d75 to
2333534
Compare
|
run buildall |
TPC-H: Total hot run time: 34227 ms |
TPC-DS: Total hot run time: 187251 ms |
ClickBench: Total hot run time: 33.03 s |
2333534 to
7eb6e74
Compare
|
run buildall |
TPC-H: Total hot run time: 34155 ms |
TPC-DS: Total hot run time: 187092 ms |
ClickBench: Total hot run time: 33.04 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
7eb6e74 to
be522a7
Compare
|
run buildall |
TPC-H: Total hot run time: 34290 ms |
TPC-DS: Total hot run time: 187204 ms |
ClickBench: Total hot run time: 32.06 s |
|
run buildall |
TPC-H: Total hot run time: 34141 ms |
TPC-DS: Total hot run time: 186382 ms |
ClickBench: Total hot run time: 33.34 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
TPC-H: Total hot run time: 34866 ms |
TPC-DS: Total hot run time: 188802 ms |
ClickBench: Total hot run time: 29.62 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
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
d135527 to
1536d0c
Compare
|
run buildall |
TPC-H: Total hot run time: 34845 ms |
TPC-DS: Total hot run time: 188444 ms |
ClickBench: Total hot run time: 30.41 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
HappenLee
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. |
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
What problem does this PR solve?
Support expression like this to be pushed down to ann index
Related PR: #54276
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)