From 0ca42fafecb4516acb51a35c94e6cc7688f97e6c Mon Sep 17 00:00:00 2001 From: Steve Carlin Date: Fri, 22 Nov 2024 09:33:02 -0800 Subject: [PATCH] IMPALA-13511: Addendum, fixed wrong case statement This was changed in code review. Because this is for the Calcite planner, the unit tests aren't in place yet. Change-Id: I12583f571392513055d73b74001a021cfc2c9813 Reviewed-on: http://gerrit.cloudera.org:8080/22098 Reviewed-by: Michael Smith Tested-by: Michael Smith --- .../apache/impala/calcite/type/ImpalaSqlIntervalQualifier.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/java/calcite-planner/src/main/java/org/apache/impala/calcite/type/ImpalaSqlIntervalQualifier.java b/java/calcite-planner/src/main/java/org/apache/impala/calcite/type/ImpalaSqlIntervalQualifier.java index da10640d8..cdeb2852d 100644 --- a/java/calcite-planner/src/main/java/org/apache/impala/calcite/type/ImpalaSqlIntervalQualifier.java +++ b/java/calcite-planner/src/main/java/org/apache/impala/calcite/type/ImpalaSqlIntervalQualifier.java @@ -71,6 +71,8 @@ public class ImpalaSqlIntervalQualifier extends SqlIntervalQualifier { case MILLISECOND: case MICROSECOND: case NANOSECOND: + break; + default: return super.evaluateIntervalLiteral(value, pos, typeSystem); }