Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/21239 )
Change subject: IMPALA-13043: Implement Join Capability to the Calcite Planner ...................................................................... Patch Set 9: (10 comments) http://gerrit.cloudera.org:8080/#/c/21239/9/java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaJoinRel.java File java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaJoinRel.java: http://gerrit.cloudera.org:8080/#/c/21239/9/java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaJoinRel.java@238 PS9, Line 238: * generates a Left Outer Join and checks the nullability of the column being output from line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/21239/9/java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaJoinRel.java@239 PS9, Line 239: * the right side of the LOJ. Since the literal TRUE is a non-null value coming into the join line too long (95 > 90) http://gerrit.cloudera.org:8080/#/c/21239/9/java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaJoinRel.java@240 PS9, Line 240: * but after the join becomes nullable, we add this function to ensure that happens. Without line too long (94 > 90) http://gerrit.cloudera.org:8080/#/c/21239/9/java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaJoinRel.java@241 PS9, Line 241: * adding this function the direct translation would be 'TRUE IS NULL' which is incorrect. line too long (92 > 90) http://gerrit.cloudera.org:8080/#/c/21239/9/java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaJoinRel.java@461 PS9, Line 461: // node. This is checked during the validatePlan(). In some cases, the check for small query line too long (96 > 90) http://gerrit.cloudera.org:8080/#/c/21239/9/java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaJoinRel.java@462 PS9, Line 462: // optimization (done at the end of the single node planning phase) will determine that the line too long (95 > 90) http://gerrit.cloudera.org:8080/#/c/21239/9/java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaJoinRel.java@463 PS9, Line 463: // cardinality of the query is lower than the exec_single_node_rows_threshold and will force line too long (96 > 90) http://gerrit.cloudera.org:8080/#/c/21239/9/java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaJoinRel.java@464 PS9, Line 464: // the num_nodes = 1. However, that check gets bypassed for some tests run through the test line too long (95 > 90) http://gerrit.cloudera.org:8080/#/c/21239/9/java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/node/ImpalaJoinRel.java@469 PS9, Line 469: (joinOp == JoinOperator.RIGHT_OUTER_JOIN || joinOp == JoinOperator.FULL_OUTER_JOIN) && line too long (94 > 90) http://gerrit.cloudera.org:8080/#/c/21239/9/java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/phys/ImpalaHashJoinNode.java File java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/phys/ImpalaHashJoinNode.java: http://gerrit.cloudera.org:8080/#/c/21239/9/java/calcite-planner/src/main/java/org/apache/impala/calcite/rel/phys/ImpalaHashJoinNode.java@44 PS9, Line 44: super(leftInput, rightInput, isStraightJoin, distMode, joinOp, equiJoinConjuncts, nonEquijoinConjuncts); line too long (108 > 90) -- To view, visit http://gerrit.cloudera.org:8080/21239 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I5db097577907d79877f52feff2922000af074ecd Gerrit-Change-Number: 21239 Gerrit-PatchSet: 9 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Comment-Date: Mon, 01 Jul 2024 20:27:15 +0000 Gerrit-HasComments: Yes
