[fix](mtmv) Mapping materialization statistics's expressionToColumnStats to mv scan plan based #36058
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
…ats to mv scan plan based (apache#35749) this is brought by apache#35436 in the method `MaterializationContext#getPlanStatistics` this get the materialization context orginal plan statistics. but the `expressionToColumnStats` in statistics is the slot of original plan. We want the statistics of original plan but the `expressionToColumnStats` in which should be mv scan plan based actually. So add the method `MaterializationContext#normalizeStatisticsColumnExpression`. when after generate the PlanStatistics in MaterializationContext, should call the normalizeStatisticsColumnExpression method.
morningman
force-pushed
the
fix_statistics_mapping_to214
branch
from
a1ba879
to
7ba35e9
Compare
|
run buildall |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed changes
commitId: f744276
pr: #35749
this is brought by #35436
in the method
MaterializationContext#getPlanStatisticsthis get the materialization context orginal plan statistics.but the
expressionToColumnStatsin statistics is the slot of original plan. We want the statistics of original plan but theexpressionToColumnStatsin which should be mv scan plan based actually. So add the methodMaterializationContext#normalizeStatisticsColumnExpression. when after generate the PlanStatistics in MaterializationContext, should call the normalizeStatisticsColumnExpression method.