-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
[chore](check) open shorten-64-to-32 error #41197
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
ed2c157
to
1a2386e
Compare
run buildall |
TPC-H: Total hot run time: 42029 ms
|
TeamCity be ut coverage result: |
TPC-DS: Total hot run time: 190643 ms
|
ClickBench: Total hot run time: 32.65 s
|
1a2386e
to
82af2cd
Compare
run buildall |
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 42143 ms
|
TPC-DS: Total hot run time: 192689 ms
|
ClickBench: Total hot run time: 31.57 s
|
82af2cd
to
4d6af96
Compare
run buildall |
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 40950 ms
|
TPC-DS: Total hot run time: 192604 ms
|
ClickBench: Total hot run time: 32.24 s
|
@@ -682,7 +684,7 @@ Status ScanLocalState<Derived>::_should_push_down_binary_predicate( | |||
DCHECK(constant_val->data == nullptr) << "constant_val should not have a value"; | |||
const auto& children = fn_call->children(); | |||
DCHECK(children.size() == 2); | |||
for (size_t i = 0; i < children.size(); i++) { |
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.
why change the code?
run buildall |
4d6af96
to
eed9b4c
Compare
run buildall |
TPC-H: Total hot run time: 41096 ms
|
TPC-DS: Total hot run time: 191202 ms
|
TeamCity be ut coverage result: |
ClickBench: Total hot run time: 33.26 s
|
run cloud_p0 |
run buildall |
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 40917 ms
|
TPC-DS: Total hot run time: 191242 ms
|
ClickBench: Total hot run time: 32.34 s
|
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. |
``` doris/be/src/pipeline/pipeline_fragment_context.cpp:1160:76: error: implicit conversion loses integer precision: 'size_type' (aka 'unsigned long') to 'int' [-Werror,-Wshorten-64-to-32] 1159 | _sink.reset(new MultiCastDataStreamSinkOperatorX( | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1160 | sink_id, sources, thrift_sink.multi_cast_stream_sink.sinks.size(), pool, | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~ ```
``` doris/be/src/pipeline/pipeline_fragment_context.cpp:1160:76: error: implicit conversion loses integer precision: 'size_type' (aka 'unsigned long') to 'int' [-Werror,-Wshorten-64-to-32] 1159 | _sink.reset(new MultiCastDataStreamSinkOperatorX( | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1160 | sink_id, sources, thrift_sink.multi_cast_stream_sink.sinks.size(), pool, | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~ ```
Proposed changes