Keep the findBestExp method exception consistent between HepPlanner and VolcanoPlanner
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 4b2d0e4..fd5687e 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 HepPlanner(
}
@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 0c22d03..80faf61 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 @@ protected void registerMaterializations() {
* query
*/
@Override public RelNode findBestExp() {
- assert root != null : "root must not be null";
+ assert root != null : "'root' must not be null";
ensureRootConverters();
registerMaterializations();
