Steve Carlin has posted comments on this change. ( http://gerrit.cloudera.org:8080/22252 )
Change subject: WIP: refactor Frontend part 1 ...................................................................... Patch Set 25: (1 comment) http://gerrit.cloudera.org:8080/#/c/22252/25/fe/src/main/java/org/apache/impala/planner/DistributedPlanner.java File fe/src/main/java/org/apache/impala/planner/DistributedPlanner.java: http://gerrit.cloudera.org:8080/#/c/22252/25/fe/src/main/java/org/apache/impala/planner/DistributedPlanner.java@562 PS25, Line 562: computeDistributionMode > computeJoinDistributionMode Thanks for the review, but this is still a work in progress. I'm not sure this is going to go in as/is, so you might want to hold off on reviewing any further. -- To view, visit http://gerrit.cloudera.org:8080/22252 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Idb8b54b8a2c36740bc6a10e8fec64dc70541327c Gerrit-Change-Number: 22252 Gerrit-PatchSet: 25 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Anonymous Coward (816) Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]> Gerrit-Comment-Date: Mon, 06 Jan 2025 16:56:13 +0000 Gerrit-HasComments: Yes
