This is an automated email from the ASF dual-hosted git repository.
berkay pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git
The following commit(s) were added to refs/heads/main by this push:
new d8405baa1d Minor: clarify comment about empty dependencies (#12786)
d8405baa1d is described below
commit d8405baa1dd910e91ec875d2ee672e04584bfd51
Author: Andrew Lamb <[email protected]>
AuthorDate: Tue Oct 8 02:49:24 2024 -0400
Minor: clarify comment about empty dependencies (#12786)
---
datafusion/physical-expr/src/equivalence/properties.rs | 2 ++
1 file changed, 2 insertions(+)
diff --git a/datafusion/physical-expr/src/equivalence/properties.rs
b/datafusion/physical-expr/src/equivalence/properties.rs
index 4ab0cac582..6a1268ef8c 100644
--- a/datafusion/physical-expr/src/equivalence/properties.rs
+++ b/datafusion/physical-expr/src/equivalence/properties.rs
@@ -1492,6 +1492,8 @@ impl<'a> DependencyEnumerator<'a> {
// Since we work on intermediate nodes, we are sure
`val.target_sort_expr`
// exists.
let target_sort_expr = node.target_sort_expr.as_ref().unwrap();
+ // An empty dependency means the referred_sort_expr represents a
global ordering.
+ // Return its projected version, which is the target_expression.
if node.dependencies.is_empty() {
return vec![vec![target_sort_expr.clone()]];
};
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
