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 <achennag...@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; }