-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[feature](meta) queries as table valued function (#) #25052
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
3b49250 to
50372cc
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
|
clang-tidy review says "All clean, LGTM! 👍" |
50372cc to
a385a28
Compare
1. Add queries view as table function. 2. Proxy result to other FEs and return merged results back to BE.
|
clang-tidy review says "All clean, LGTM! 👍" |
1. Add queries view as table function. 2. Proxy result to other FEs and return merged results back to BE.
a385a28 to
18b3ac0
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
yiguolei
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
|
run buildall |
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
hf200012
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
|
TeamCity be ut coverage result: |
|
(From new machine)TeamCity pipeline, clickbench performance test result: |
1. Add queries view as table function. 2. Proxy result to other FEs and return merged results back to BE.
18b3ac0 to
e0c5eab
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
|
run buildall |
|
TeamCity be ut coverage result: |
|
(From new machine)TeamCity pipeline, clickbench performance test result: |
1. Add queries view as table function. 2. Proxy result to other FEs and return merged results back to BE.
e0c5eab to
3e63ca7
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
1. Add queries view as table function. 2. Proxy result to other FEs and return merged results back to BE.
3e63ca7 to
2fd275e
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
2fd275e to
4d4dd10
Compare
1. Add queries view as table function. 2. Proxy result to other FEs and return merged results back to BE.
|
clang-tidy review says "All clean, LGTM! 👍" |
1. Add queries view as table function. 2. Proxy result to other FEs and return merged results back to BE.
4d4dd10 to
3836ee0
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
3836ee0 to
4d0c4cb
Compare
1. Add queries view as table function. 2. Proxy result to other FEs and return merged results back to BE.
|
clang-tidy review says "All clean, LGTM! 👍" |
1. Add queries view as table function. 2. Proxy result to other FEs and return merged results back to BE.
4d0c4cb to
c0a62fc
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
|
clang-tidy review says "All clean, LGTM! 👍" |
1. Add queries view as table function. 2. Proxy result to other FEs and return merged results back to BE.
c0a62fc to
42cd5b0
Compare
|
clang-tidy review says "All clean, LGTM! 👍" |
|
run buildall |
|
run buildall |
|
clang-tidy review says "All clean, LGTM! 👍" |
|
TeamCity be ut coverage result: |
|
(From new machine)TeamCity pipeline, clickbench performance test result: |
yiguolei
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
…he#25052) 1. Add queries view as table function. 2. Proxy result to other FEs and return merged results back to BE. Co-authored-by: yiguolei <676222867@qq.com>
Proposed changes
Issue Number: close #25051