Expression is stateful to store the result of evaluation, so we can't… (

#4190)

* Expression is stateful to store the result of evaluation, so we can't share it inter threads.

* Fix defef nullptr.

src/storage/query/ScanEdgeProcessor.cpp

@@ -101,8 +101,14 @@ StoragePlan<Cursor> ScanEdgeProcessor::buildPlan(
  edges.emplace_back(
  std::make_unique<FetchEdgeNode>(context, &edgeContext_, ec.first, &ec.second));
  }
- auto output = std::make_unique<ScanEdgePropNode>(
- context, std::move(edges), enableReadFollower_, limit_, cursors, result, expCtx, filter_);
+ auto output = std::make_unique<ScanEdgePropNode>(context,
+ std::move(edges),
+ enableReadFollower_,
+ limit_,
+ cursors,
+ result,
+ expCtx,
+ filter_ == nullptr ? nullptr : filter_->clone());
 
  plan.addNode(std::move(output));
  return plan;

src/storage/query/ScanVertexProcessor.cpp

@@ -104,8 +104,15 @@ StoragePlan<Cursor> ScanVertexProcessor::buildPlan(
  for (const auto& tc : tagContext_.propContexts_) {
  tags.emplace_back(std::make_unique<TagNode>(context, &tagContext_, tc.first, &tc.second));
  }
- auto output = std::make_unique<ScanVertexPropNode>(
- context, std::move(tags), enableReadFollower_, limit_, cursors, result, expCtx, filter_);
+ auto output =
+ std::make_unique<ScanVertexPropNode>(context,
+ std::move(tags),
+ enableReadFollower_,
+ limit_,
+ cursors,
+ result,
+ expCtx,
+ filter_ == nullptr ? nullptr : filter_->clone());
 
  plan.addNode(std::move(output));
  return plan;