Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/14216 )
Change subject: WIP: IMPALA-2138,IMPALA-1306: project tuples before exchanges ...................................................................... Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/14216/8/fe/src/main/java/org/apache/impala/planner/ProjectionPass.java File fe/src/main/java/org/apache/impala/planner/ProjectionPass.java: http://gerrit.cloudera.org:8080/#/c/14216/8/fe/src/main/java/org/apache/impala/planner/ProjectionPass.java@288 PS8, Line 288: SlotDescriptor projectedSlotDesc = analyzer.copySlotDescriptor(slotDesc, projectedTuple); line too long (95 > 90) http://gerrit.cloudera.org:8080/#/c/14216/8/fe/src/main/java/org/apache/impala/planner/ProjectionPass.java@296 PS8, Line 296: TreeNode.collect(exprs, Predicates.instanceOf(TupleIsNullPredicate.class), tupleIsNullPreds); line too long (97 > 90) -- To view, visit http://gerrit.cloudera.org:8080/14216 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I94ccf2d1acecd9a593f4c29fc15202a799d2f7f5 Gerrit-Change-Number: 14216 Gerrit-PatchSet: 8 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Comment-Date: Mon, 25 Jan 2021 19:52:49 +0000 Gerrit-HasComments: Yes
