Skip to content
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](mtmv) Fix some pr to 21, prs are (#39041)(#38958)(#39541) #39678

Merged
merged 5 commits into from
Aug 22, 2024

Conversation

seawinde
Copy link
Contributor

Proposed changes

pr: #39041
commitId: 2256298

pr: #38958
commitId: c365cb6

pr: #39541
commitId: 89bb669

… group by (apache#39041)

this is brought by apache#34185
if query external table without group by, rewrite by materialized view
will fail
such as mv def is

            select  o_custkey, o_orderdate
            from ${hive_catalog_name}.${hive_database}.${hive_table};

query is query external table directly as following, this would fail
when try to rewrte by materialized view

            select o_custkey
            from ${hive_catalog_name}.${hive_database}.${hive_table};

this pr fix the problem.
…terialized view (apache#38958)

This is brought by apache#34274

if mv def is
select o_orderdate from  orders group by o_orderdate;

query is as followiing, the result is wrong.
select 1 from orders group by o_orderdate;
…an for performance (apache#39541)

Before query rewrite by materialized view, we collecet the table which
query used by method org.apache.doris.mtmv.MTMVCache#from.

In MTMVCache#from we calcute the cost of plan which is useless for
collecting table.
So add boolean needCost param in method MTMVCache#from to identify
that if need cost of plan or not for performance.
@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@seawinde
Copy link
Contributor Author

run buildall

@seawinde
Copy link
Contributor Author

run buildall

@seawinde
Copy link
Contributor Author

run buildall

@yiguolei yiguolei merged commit 021982f into apache:branch-2.1 Aug 22, 2024
20 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants