Steve Carlin has posted comments on this change. ( http://gerrit.cloudera.org:8080/24781 )
Change subject: IMPALA-15312: Say when HBO changed a join's build side ...................................................................... Patch Set 2: I'm a little worried that this is too specific. Yes, invert joins can affect the plan. But lots of other things can too. The initial join order is set up based on cardinalities and it would be impossible to show how join order affects them. I always view this "invert join" step as a correction step. "We didn't get it right the first time, so we're just fixing it." To put emphasis on this during a plan seems a bit off to me. -- To view, visit http://gerrit.cloudera.org:8080/24781 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifc9517936c3f93865917a5f3fa3924c641374fcd Gerrit-Change-Number: 24781 Gerrit-PatchSet: 2 Gerrit-Owner: Aleksandr Efimov <[email protected]> Gerrit-Reviewer: Aleksandr Efimov <[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: Steve Carlin <[email protected]> Gerrit-Comment-Date: Sat, 05 Sep 2026 20:44:35 +0000 Gerrit-HasComments: No
