xavlee opened a new pull request, #19246: URL: https://github.com/apache/datafusion/pull/19246
Which issue does this PR close? Closes #19238 Rationale for this change SortMergeJoinExec is currently displayed inconsistently across physical plan formats. The tree-format plan uses SortMergeJoinExec, while the indent-format plan uses SortMergeJoin. This inconsistency leads to confusing output and mismatched expectations in SQL logic tests. Standardizing the operator name improves clarity and aligns with the naming conventions of other execution operators. What changes are included in this PR? Updates the DisplayAs implementation for SortMergeJoinExec so both the indent and tree physical plan formats display the same operator name. Updates SQL Logic Test expected outputs in joins.slt to reflect the unified naming. No functional behavior changes; this is a display/consistency fix. Are these changes tested? Yes. Existing SQL Logic Tests reference the operator name in physical plan output. I updated those expected outputs to match the new standardized naming. All tests pass with the updated format. Are there any user-facing changes? Yes—users inspecting physical plans will now consistently see: SortMergeJoinExec instead of an inconsistent mix of SortMergeJoin and SortMergeJoinExec. This is a non-breaking, display-only change. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
