Skip to content

Commit d8405ba

Browse files
authored
Minor: clarify comment about empty dependencies (#12786)
1 parent 577e4bb commit d8405ba

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

datafusion/physical-expr/src/equivalence/properties.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1492,6 +1492,8 @@ impl<'a> DependencyEnumerator<'a> {
14921492
// Since we work on intermediate nodes, we are sure `val.target_sort_expr`
14931493
// exists.
14941494
let target_sort_expr = node.target_sort_expr.as_ref().unwrap();
1495+
// An empty dependency means the referred_sort_expr represents a global ordering.
1496+
// Return its projected version, which is the target_expression.
14951497
if node.dependencies.is_empty() {
14961498
return vec![vec![target_sort_expr.clone()]];
14971499
};

0 commit comments

Comments
 (0)