Steve Carlin has posted comments on this change. ( http://gerrit.cloudera.org:8080/23560 )
Change subject: IMPALA-14421: Calcite planner: case statement returning wrong types for char, varchar ...................................................................... Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/23560/5/java/calcite-planner/src/main/java/org/apache/impala/calcite/coercenodes/CoerceOperandShuttle.java File java/calcite-planner/src/main/java/org/apache/impala/calcite/coercenodes/CoerceOperandShuttle.java: http://gerrit.cloudera.org:8080/#/c/23560/5/java/calcite-planner/src/main/java/org/apache/impala/calcite/coercenodes/CoerceOperandShuttle.java@49 PS5, Line 49: import com.google.common.collect.Lists; > Duplicate import. Already imported at L23. Done http://gerrit.cloudera.org:8080/#/c/23560/5/java/calcite-planner/src/main/java/org/apache/impala/calcite/functions/FunctionResolver.java File java/calcite-planner/src/main/java/org/apache/impala/calcite/functions/FunctionResolver.java: http://gerrit.cloudera.org:8080/#/c/23560/5/java/calcite-planner/src/main/java/org/apache/impala/calcite/functions/FunctionResolver.java@189 PS5, Line 189: : private static Function getSpecialProcessingFunction(String lowercaseName, : List<Type> impalaArgTypes, boolean exactMatch) { > Please add documentation for this function. Done http://gerrit.cloudera.org:8080/#/c/23560/5/testdata/workloads/functional-query/queries/QueryTest/calcite.test File testdata/workloads/functional-query/queries/QueryTest/calcite.test: http://gerrit.cloudera.org:8080/#/c/23560/5/testdata/workloads/functional-query/queries/QueryTest/calcite.test@1136 PS5, Line 1136: chars_tiny > functional.chars_tiny Done -- To view, visit http://gerrit.cloudera.org:8080/23560 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I82d657f4bfce432c458ee8198188dadf9f23f2ef Gerrit-Change-Number: 23560 Gerrit-PatchSet: 5 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Fang-Yu Rao <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]> Gerrit-Reviewer: Pranav Lodha <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]> Gerrit-Comment-Date: Sun, 16 Nov 2025 22:17:15 +0000 Gerrit-HasComments: Yes
