[hive] branch master updated: HIVE-26426: StringIndexOutOfBoundsException in CalcitePlanner#canCBOHandleAst (Abhay Chennagiri reviewed by John Sherman, Stamatis Zampetakis)

[za...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 186fb0d85af HIVE-26426: StringIndexOutOfBoundsException in CalcitePlanner#canCBOHandleAst (Abhay Chennagiri reviewed by John Sherman, Stamatis Zampetakis)
186fb0d85af is described below

commit 186fb0d85af63b61bc10ba5372e35895754b1a6a
Author: Abhay Chennagiri <ac...@cloudera.com>
AuthorDate: Fri Jul 22 19:14:30 2022 -0700

    HIVE-26426: StringIndexOutOfBoundsException in CalcitePlanner#canCBOHandleAst (Abhay Chennagiri reviewed by John Sherman, Stamatis Zampetakis)
    
    Closes #3474
---
 ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java b/ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java
index 765e2e46463..79dc618a541 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java
@@ -950,7 +950,7 @@ public class CalcitePlanner extends SemanticAnalyzer {
     if (msg == null) {
       return Pair.of(true, msg);
     }
-    msg = msg.substring(0, msg.length() - 2);
+
     if (needToLogMessage) {
       STATIC_LOG.info("Not invoking CBO because the statement " + msg);
     }
@@ -1006,10 +1006,10 @@ public class CalcitePlanner extends SemanticAnalyzer {
       if (queryProperties.hasLateralViews()) {
         msg += "has lateral views; ";
       }
-
       if (msg.isEmpty()) {
         msg += "has some unspecified limitations; ";
       }
+      msg = msg.substring(0, msg.length() - 2);
     }
     return msg;
   }