Skip to content

Commit

Permalink
make sort plan node format as external merge sort
Browse files Browse the repository at this point in the history
  • Loading branch information
xx01cyx committed Oct 21, 2024
1 parent 3f258ad commit 96d4f4f
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/execution/fmt_impl.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,9 @@ auto UpdatePlanNode::PlanNodeToString() const -> std::string {
}

auto SortPlanNode::PlanNodeToString() const -> std::string {
return fmt::format("Sort {{ order_bys={} }}", order_bys_);
// Note(f24): A sort plan node will be converted to an external merge sort executor in
// Fall 2024. So `ExternalMergeSort` is returned instead of `Sort`.
return fmt::format("ExternalMergeSort {{ order_bys={} }}", order_bys_);
}

auto LimitPlanNode::PlanNodeToString() const -> std::string { return fmt::format("Limit {{ limit={} }}", limit_); }
Expand Down

0 comments on commit 96d4f4f

Please sign in to comment.