(impala) branch master updated: IMPALA-13511: Addendum, fixed wrong case statement

[michaelsmith]

This is an automated email from the ASF dual-hosted git repository.

michaelsmith pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git


The following commit(s) were added to refs/heads/master by this push:
     new 0ca42fafe IMPALA-13511: Addendum, fixed wrong case statement
0ca42fafe is described below

commit 0ca42fafecb4516acb51a35c94e6cc7688f97e6c
Author: Steve Carlin <scar...@cloudera.com>
AuthorDate: Fri Nov 22 09:33:02 2024 -0800

    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 <michael.sm...@cloudera.com>
    Tested-by: Michael Smith <michael.sm...@cloudera.com>
---
 .../java/org/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);
     }