(calcite) branch main updated: Keep the findBestExp method exception consistent between HepPlanner and VolcanoPlanner

[xiong]

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

xiong pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/calcite.git


The following commit(s) were added to refs/heads/main by this push:
     new 50fa89469b Keep the findBestExp method exception consistent between 
HepPlanner and VolcanoPlanner
50fa89469b is described below

commit 50fa89469b56ba211585d6f60bf6166507b0b1bc
Author: ChengJie1053 <18033291...@163.com>
AuthorDate: Mon Aug 19 10:57:09 2024 +0800

    Keep the findBestExp method exception consistent between HepPlanner and 
VolcanoPlanner
---
 core/src/main/java/org/apache/calcite/plan/hep/HepPlanner.java        | 4 ++--
 .../src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/core/src/main/java/org/apache/calcite/plan/hep/HepPlanner.java 
b/core/src/main/java/org/apache/calcite/plan/hep/HepPlanner.java
index 4b2d0e4a8a..fd5687e40c 100644
--- a/core/src/main/java/org/apache/calcite/plan/hep/HepPlanner.java
+++ b/core/src/main/java/org/apache/calcite/plan/hep/HepPlanner.java
@@ -186,14 +186,14 @@ public class HepPlanner extends AbstractRelOptPlanner {
   }
 
   @Override public RelNode findBestExp() {
-    requireNonNull(root, "root");
+    requireNonNull(root, "'root' must not be null");
 
     executeProgram(mainProgram);
 
     // Get rid of everything except what's in the final plan.
     collectGarbage();
     dumpRuleAttemptsInfo();
-    return buildFinalPlan(requireNonNull(root, "root"));
+    return buildFinalPlan(requireNonNull(root, "'root' must not be null"));
   }
 
   /** Top-level entry point for a program. Initializes state and then invokes
diff --git 
a/core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java 
b/core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java
index 0c22d03820..80faf61d92 100644
--- a/core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java
+++ b/core/src/main/java/org/apache/calcite/plan/volcano/VolcanoPlanner.java
@@ -517,7 +517,7 @@ public class VolcanoPlanner extends AbstractRelOptPlanner {
    * query
    */
   @Override public RelNode findBestExp() {
-    assert root != null : "root must not be null";
+    assert root != null : "'root' must not be null";
     ensureRootConverters();
     registerMaterializations();