-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[fix](Nereids) mv in select materialized_view should disable show table #24104
Conversation
run buildall |
(From new machine)TeamCity pipeline, clickbench performance test result: |
1ad4f4a
to
a31d1f4
Compare
run buildall |
(From new machine)TeamCity pipeline, clickbench performance test result: |
a31d1f4
to
4723fe6
Compare
run buildall |
(From new machine)TeamCity pipeline, clickbench performance test result: |
4723fe6
to
22b72fe
Compare
run buildall |
(From new machine)TeamCity pipeline, clickbench performance test result: |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
…le (#24104) mv in select materialized_view should disable show table, because Nereids planner can output the string such as slot#[0] in toSql() of SlotRef. Note this is only a temporary solution, will use an expression translator later
…le (#24104) mv in select materialized_view should disable show table, because Nereids planner can output the string such as slot#[0] in toSql() of SlotRef. Note this is only a temporary solution, will use an expression translator later
Proposed changes
mv in select materialized_view should disable show table,
because Nereids planner can output the string such as
slot#[0]
intoSql()
ofSlotRef
. Note this is only atemporary solution, will use an expression translator later
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...