Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/15096 )
Change subject: WIP: IMPALA-9156: share broadcast join builds ...................................................................... Patch Set 8: (3 comments) http://gerrit.cloudera.org:8080/#/c/15096/8/fe/src/main/java/org/apache/impala/planner/PlanFragment.java File fe/src/main/java/org/apache/impala/planner/PlanFragment.java: http://gerrit.cloudera.org:8080/#/c/15096/8/fe/src/main/java/org/apache/impala/planner/PlanFragment.java@201 PS8, Line 201: DataStreamSink streamSink = new DataStreamSink((ExchangeNode)destNode_, outputPartition_); line too long (96 > 90) http://gerrit.cloudera.org:8080/#/c/15096/8/fe/src/main/java/org/apache/impala/planner/PlanFragment.java@371 PS8, Line 371: // join. ParallelPlanner sets the destination fragment when adding the JoinBuildSink. line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/15096/8/fe/src/main/java/org/apache/impala/planner/PlanFragment.java@400 PS8, Line 400: // join. ParallelPlanner sets the destination fragment when adding the JoinBuildSink. line too long (91 > 90) -- To view, visit http://gerrit.cloudera.org:8080/15096 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4c67e4b2c87ed0fba648f1e1710addb885d66dc7 Gerrit-Change-Number: 15096 Gerrit-PatchSet: 8 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Bikramjeet Vig <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Wed, 26 Feb 2020 23:53:32 +0000 Gerrit-HasComments: Yes
