Zoltan Borok-Nagy has posted comments on this change. ( http://gerrit.cloudera.org:8080/22032 )
Change subject: IMPALA-13086: Lower AggregationNode estimate using stats predicate ...................................................................... Patch Set 16: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/22032/16//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/22032/16//COMMIT_MSG@29 PS16, Line 29: Therefore, the UnionNode will be : acknowledged as the first producer of that TupleId. Also, the estimates corresponding to such Iceberg V2 tables won't be affected by this patch. AFAICT it won't be trivial to resolve this because UnionNode doesn't know the predicates and the individual ScanNodes have their own cardinalities. I.e. cardinality information should come from the UnionNode, but ScanNodes have the predicates. -- To view, visit http://gerrit.cloudera.org:8080/22032 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ia840d68f1c4f126d4e928461ec5c44545dbf25f8 Gerrit-Change-Number: 22032 Gerrit-PatchSet: 16 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Comment-Date: Tue, 07 Jan 2025 11:33:22 +0000 Gerrit-HasComments: Yes
