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

csringhofer 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 83036b13e IMPALA-14487: Calcite planner: handle escaped double quote 
character
83036b13e is described below

commit 83036b13e5ff5531444af91fd9e46a05d5547455
Author: Steve Carlin <[email protected]>
AuthorDate: Mon Sep 22 12:35:29 2025 -0700

    IMPALA-14487: Calcite planner: handle escaped double quote character
    
    Added the double quote character to the fix added for IMPALA-13525 to
    handle escaped characters.
    
    Change-Id: Ic65fbb4546ae071a9442c0b4884254c15b268087
    Reviewed-on: http://gerrit.cloudera.org:8080/23695
    Reviewed-by: Michael Smith <[email protected]>
    Reviewed-by: Peter Rozsa <[email protected]>
    Tested-by: Impala Public Jenkins <[email protected]>
---
 .../src/main/java/org/apache/impala/calcite/util/ParserUtil.java     | 1 +
 testdata/workloads/functional-query/queries/QueryTest/calcite.test   | 5 +++++
 2 files changed, 6 insertions(+)

diff --git 
a/java/calcite-planner/src/main/java/org/apache/impala/calcite/util/ParserUtil.java
 
b/java/calcite-planner/src/main/java/org/apache/impala/calcite/util/ParserUtil.java
index c3275af7c..bf034bd4d 100644
--- 
a/java/calcite-planner/src/main/java/org/apache/impala/calcite/util/ParserUtil.java
+++ 
b/java/calcite-planner/src/main/java/org/apache/impala/calcite/util/ParserUtil.java
@@ -98,6 +98,7 @@ public class ParserUtil {
         case '{':
         case '}':
         case '|':
+        case '"':
           builder.append(nextChar);
           i++;
           break;
diff --git a/testdata/workloads/functional-query/queries/QueryTest/calcite.test 
b/testdata/workloads/functional-query/queries/QueryTest/calcite.test
index 3723375b4..ab3f04be0 100644
--- a/testdata/workloads/functional-query/queries/QueryTest/calcite.test
+++ b/testdata/workloads/functional-query/queries/QueryTest/calcite.test
@@ -1187,3 +1187,8 @@ char, char, varchar
 'NULL','b                                                                      
                                                                     ','NULL'
 'NULL','NULL','NULL'
 ====
+---- QUERY
+select '\"';
+---- RESULTS
+'"'
+====

Reply via email to