Tim Armstrong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14790 )
Change subject: IMPALA-9126: part 4: hash join builder manages spilling ...................................................................... Patch Set 8: (6 comments) http://gerrit.cloudera.org:8080/#/c/14790/8/be/src/exec/partitioned-hash-join-builder.h File be/src/exec/partitioned-hash-join-builder.h: http://gerrit.cloudera.org:8080/#/c/14790/8/be/src/exec/partitioned-hash-join-builder.h@200 PS8, Line 200: when *repartitioning as true > I think this is supposed to be "when *repartitioned is true"? Done http://gerrit.cloudera.org:8080/#/c/14790/8/be/src/exec/partitioned-hash-join-builder.h@205 PS8, Line 205: /// 'output_partitions' for build modes like right outer join that output unmatched rows. > line too long (91 > 90) Done http://gerrit.cloudera.org:8080/#/c/14790/8/be/src/exec/partitioned-hash-join-builder.h@213 PS8, Line 213: repartitioning > same - repartitioned? Done http://gerrit.cloudera.org:8080/#/c/14790/8/be/src/exec/partitioned-hash-join-builder.cc File be/src/exec/partitioned-hash-join-builder.cc: http://gerrit.cloudera.org:8080/#/c/14790/8/be/src/exec/partitioned-hash-join-builder.cc@641 PS8, Line 641: Pick the next spilled partition the input partition will stay in : // 'spilled_partitions_' until we are done probing it or repartitioning its probe. > Having some trouble parsing this sentence. I think you missed a period afte Done http://gerrit.cloudera.org:8080/#/c/14790/8/be/src/exec/partitioned-hash-join-node.cc File be/src/exec/partitioned-hash-join-node.cc: http://gerrit.cloudera.org:8080/#/c/14790/8/be/src/exec/partitioned-hash-join-node.cc@1091 PS8, Line 1091: // Walk the partitions that had hash tables built for the probe phase and close them. This was also stale http://gerrit.cloudera.org:8080/#/c/14790/8/be/src/exec/partitioned-hash-join-node.cc@1098 PS8, Line 1098: // The build partitions we need to retain for further processing. > stale Done -- To view, visit http://gerrit.cloudera.org:8080/14790 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6de5f62e3eacf80f72c8ea0ed8cba012f0f53c90 Gerrit-Change-Number: 14790 Gerrit-PatchSet: 8 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Bikramjeet Vig <bikramjeet....@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Comment-Date: Thu, 05 Dec 2019 20:59:45 +0000 Gerrit-HasComments: Yes