-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[featrue](expr) check expr return type and block column type #35032
base: master
Are you sure you want to change the base?
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: |
run buildall |
TeamCity be ut coverage result: |
5ded43d
to
45c3578
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 39992 ms
|
TPC-DS: Total hot run time: 169337 ms
|
ClickBench: Total hot run time: 30.9 s
|
run buildall |
TPC-H: Total hot run time: 42010 ms
|
TPC-DS: Total hot run time: 168768 ms
|
ClickBench: Total hot run time: 30.68 s
|
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 40880 ms
|
TPC-DS: Total hot run time: 172460 ms
|
ClickBench: Total hot run time: 30.69 s
|
00c3324
to
9dc8398
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 40265 ms
|
TPC-DS: Total hot run time: 171601 ms
|
TeamCity be ut coverage result: |
ClickBench: Total hot run time: 30.34 s
|
be/src/vec/exprs/vexpr_context.cpp
Outdated
"column is nullable = {} , " | ||
"error type msg = {}", | ||
data.name, expr_data_type->get_name(), column_data_type->get_name(), | ||
column_ptr->is_nullable(), st.msg()); |
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.
column_ptr->dump_structure()
9dc8398
to
cd04324
Compare
TPC-H: Total hot run time: 42094 ms
|
TPC-DS: Total hot run time: 169796 ms
|
ClickBench: Total hot run time: 30.24 s
|
7cb3c8b
to
895c383
Compare
run buildall |
TPC-H: Total hot run time: 42496 ms
|
TeamCity be ut coverage result: |
TPC-DS: Total hot run time: 171946 ms
|
ClickBench: Total hot run time: 30.23 s
|
895c383
to
c153f6d
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 41515 ms
|
TPC-DS: Total hot run time: 170987 ms
|
ClickBench: Total hot run time: 30.53 s
|
TeamCity be ut coverage result: |
c153f6d
to
4d20c93
Compare
4d20c93
to
1d29209
Compare
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
clang-tidy review says "All clean, LGTM! 👍" |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 41874 ms
|
TPC-DS: Total hot run time: 173524 ms
|
TeamCity be ut coverage result: |
ClickBench: Total hot run time: 30.37 s
|
Proposed changes
Issue Number: close #xxx
Further comments
If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...