Aleksandr Efimov has uploaded this change for review. ( http://gerrit.cloudera.org:8080/24564
Change subject: IMPALA-15156: Use stripped rels for cost metadata ...................................................................... IMPALA-15156: Use stripped rels for cost metadata CTE and materialized-view rewrites can cost RelSubset inputs directly. Impala's non-cumulative cost model asks those inputs for row-count and size metadata, but size metadata should be read from the selected or original rel instead of from the RelSubset wrapper. Move the unwrap into ImpalaRelMdNonCumulativeCost and remove the temporary RelSubset average-column-size metadata override. Add a focused unit test for the join-cost path. Testing: - git diff --check - Focused calcite-planner JUnit: TestImpalaRelMdNonCumulativeCost Change-Id: Ide226449f7b7a33fffd67f403b03fd5b2fee53ee --- M java/calcite-planner/src/main/java/org/apache/impala/calcite/schema/ImpalaRelMdNonCumulativeCost.java M java/calcite-planner/src/main/java/org/apache/impala/calcite/schema/ImpalaRelMdSize.java A java/calcite-planner/src/test/java/org/apache/impala/calcite/schema/TestImpalaRelMdNonCumulativeCost.java 3 files changed, 142 insertions(+), 15 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/64/24564/1 -- To view, visit http://gerrit.cloudera.org:8080/24564 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ide226449f7b7a33fffd67f403b03fd5b2fee53ee Gerrit-Change-Number: 24564 Gerrit-PatchSet: 1 Gerrit-Owner: Aleksandr Efimov <[email protected]>
