Print fields of plan node when null (#3805)

vesoft-inc · Feb 14, 2022 · ed16669 · ed16669
1 parent 5a4be98
commit ed16669
Showing 1 changed file with 4 additions and 12 deletions.
diff --git a/src/graph/planner/plan/Query.cpp b/src/graph/planner/plan/Query.cpp
@@ -734,15 +734,9 @@ void Traverse::cloneMembers(const Traverse& g) {
 
 std::unique_ptr<PlanNodeDescription> Traverse::explain() const {
  auto desc = GetNeighbors::explain();
- if (range_ != nullptr) {
- addDescription("steps", range_->toString(), desc.get());
- }
- if (vFilter_ != nullptr) {
- addDescription("vertex filter", vFilter_->toString(), desc.get());
- }
- if (eFilter_ != nullptr) {
- addDescription("edge filter", eFilter_->toString(), desc.get());
- }
+ addDescription("steps", range_ != nullptr ? range_->toString() : "", desc.get());
+ addDescription("vertex filter", vFilter_ != nullptr ? vFilter_->toString() : "", desc.get());
+ addDescription("edge filter", eFilter_ != nullptr ? eFilter_->toString() : "", desc.get());
  addDescription("if_track_previous_path", util::toJson(trackPrevPath_), desc.get());
  return desc;
 }
@@ -770,9 +764,7 @@ void AppendVertices::cloneMembers(const AppendVertices& a) {
 
 std::unique_ptr<PlanNodeDescription> AppendVertices::explain() const {
  auto desc = GetVertices::explain();
- if (vFilter_ != nullptr) {
- addDescription("vertex_filter", vFilter_->toString(), desc.get());
- }
+ addDescription("vertex_filter", vFilter_ != nullptr ? vFilter_->toString() : "", desc.get());
  addDescription("if_track_previous_path", util::toJson(trackPrevPath_), desc.get());
  return desc;
 }