fix: store the pointer in init_check (#555)

* fix: store the pointer in init_check

Signed-off-by: Alex Chi <[email protected]>

* fix

Signed-off-by: Alex Chi <[email protected]>

---------

Signed-off-by: Alex Chi <[email protected]>

Author: skyzh

src/execution/init_check_executor.cpp

@@ -15,9 +15,9 @@
 
 namespace bustub {
 
-InitCheckExecutor::InitCheckExecutor(ExecutorContext *exec_ctx, const AbstractPlanNodeRef &plan,
+InitCheckExecutor::InitCheckExecutor(ExecutorContext *exec_ctx, AbstractPlanNodeRef plan,
                                      std::unique_ptr<AbstractExecutor> &&child_executor)
-    : AbstractExecutor{exec_ctx}, plan_{plan}, child_executor_{std::move(child_executor)} {}
+    : AbstractExecutor{exec_ctx}, plan_{std::move(plan)}, child_executor_{std::move(child_executor)} {}
 
 void InitCheckExecutor::Init() {
   if (!child_executor_) {

src/include/execution/executors/init_check_executor.h

@@ -32,7 +32,7 @@ class InitCheckExecutor : public AbstractExecutor {
    * @param plan The init check plan to be executed
    * @param child_executor The child executor from which init calls are counted
    */
-  InitCheckExecutor(ExecutorContext *exec_ctx, const AbstractPlanNodeRef &plan,
+  InitCheckExecutor(ExecutorContext *exec_ctx, AbstractPlanNodeRef plan,
                     std::unique_ptr<AbstractExecutor> &&child_executor);
 
   /** Initialize the InitCheck */
@@ -63,7 +63,7 @@ class InitCheckExecutor : public AbstractExecutor {
   constexpr static const bool EXECUTOR_EXHAUSTED{false};
 
   /** The init check plan node to be executed */
-  const AbstractPlanNodeRef &plan_;
+  AbstractPlanNodeRef plan_;
 
   /** The child executor from which tuples are obtained */
   std::unique_ptr<AbstractExecutor> child_executor_;