-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[opt](lambda) let lambda expression support refer outer slot #44365
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:
|
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/SubExprAnalyzer.java
Outdated
Show resolved
Hide resolved
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/SubExprAnalyzer.java
Show resolved
Hide resolved
c12ee00 to
fb6e74e
Compare
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/SubExprAnalyzer.java
Show resolved
Hide resolved
|
run buildall |
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.
the code compile failed, could check it
and maybe could add some desc to explain want to do in this PR
|
run buildall |
1 similar comment
|
run buildall |
|
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 40130 ms |
TPC-DS: Total hot run time: 199281 ms |
ClickBench: Total hot run time: 32.79 s |
|
run buildall |
TPC-H: Total hot run time: 39989 ms |
TPC-DS: Total hot run time: 197476 ms |
|
TeamCity be ut coverage result: |
ClickBench: Total hot run time: 33.06 s |
|
run buildall |
|
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 40206 ms |
TPC-DS: Total hot run time: 197854 ms |
ClickBench: Total hot run time: 32.32 s |
2ae4cac to
f94bfa9
Compare
|
run buildall |
TPC-H: Total hot run time: 40116 ms |
TPC-DS: Total hot run time: 197608 ms |
ClickBench: Total hot run time: 32.83 s |
|
TeamCity be ut coverage result: |
|
run buildall |
|
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 40160 ms |
TPC-DS: Total hot run time: 197556 ms |
ClickBench: Total hot run time: 33.55 s |
|
run p0 |
…#44365) ```sql CREATE TABLE `array_test` ( `id` int(11) NULL COMMENT "", `c_array` ARRAY<int(11)> NULL COMMENT "" ) ENGINE=OLAP DUPLICATE KEY(`id`) COMMENT "OLAP" DISTRIBUTED BY HASH(`id`) BUCKETS 1 PROPERTIES ( "replication_allocation" = "tag.location.default: 1", "in_memory" = "false", "storage_format" = "V2" ); ``` this sql is ok ```sql select *, array_map(x -> x+1, c_array) from array_test ``` but lambda expressions in `array_map` do not support field arguments for example: ```sql select *, array_map(x -> x+id, c_array) from array_test ``` This pr solves two problems 1. support column arg in lambda expression 2. prevent high memory usage due to large block data Related PR: #xxx let lambda expression support refer outer slot Co-authored-by: garenshi <garenshi@tencent.com>
What problem does this PR solve?
this sql is ok
but lambda expressions in
array_mapdo not support field argumentsfor example:
This pr solves two problems
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)