From 740fc55a66c7c740150cd6a3ad17f3064a05006c Mon Sep 17 00:00:00 2001
From: Yee <2520865+yixinglu@users.noreply.github.com>
Date: Fri, 5 Mar 2021 17:11:31 +0800
Subject: [PATCH] Fix explain validator subplan generation (#781)
Co-authored-by: jie.wang <38901892+jievince@users.noreply.github.com>
---
src/validator/ExplainValidator.cpp | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/src/validator/ExplainValidator.cpp b/src/validator/ExplainValidator.cpp
index 1910e3d2..ef20f8c3 100644
--- a/src/validator/ExplainValidator.cpp
+++ b/src/validator/ExplainValidator.cpp
@@ -66,11 +66,9 @@ Status ExplainValidator::validateImpl() {
}
Status ExplainValidator::toPlan() {
- auto subPlanStatus = Planner::toPlan(validator_->getAstContext());
- NG_RETURN_IF_ERROR(subPlanStatus);
- auto subPlan = std::move(subPlanStatus).value();
- root_ = subPlan.root;
- tail_ = subPlan.tail;
+ // The execution plan has been generated in validateImpl function
+ root_ = validator_->root();
+ tail_ = validator_->tail();
VLOG(1) << "root: " << root_->kind() << " tail: " << tail_->kind();
return Status::OK();
}
--
GitLab