geoffreyclaude commented on code in PR #26118:
URL: https://github.com/apache/datafusion/pull/26118#discussion_r4375693724
##########
datafusion/physical-plan/src/execution_plan.rs:
##########
@@ -233,6 +233,10 @@ pub trait ExecutionPlan: Any + Debug + DisplayAs + Send +
Sync {
/// DataFusion uses this metadata to apply certain optimizations
/// such as automatically repartitioning correctly.
///
+ /// This describes row sequence, not how input expressions map to outputs.
+ /// Custom nodes that support sort pushdown should implement
+ /// [`Self::try_pushdown_sort`]; returning `true` here alone does not opt
in.
Review Comment:
I've gone with the validated fallback, so I’ve removed the upgrade note.
--
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]
