This is an automated email from the ASF dual-hosted git repository.
comphead 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 2c2e0e70e8 Simplifications (#12656)
2c2e0e70e8 is described below
commit 2c2e0e70e84cf2d8a08aaba5bcd99c1720f3e0f9
Author: Mustafa Akur <[email protected]>
AuthorDate: Tue Oct 1 09:37:50 2024 -0700
Simplifications (#12656)
---
.../physical-plan/src/windows/bounded_window_agg_exec.rs | 16 +++++-----------
1 file changed, 5 insertions(+), 11 deletions(-)
diff --git a/datafusion/physical-plan/src/windows/bounded_window_agg_exec.rs
b/datafusion/physical-plan/src/windows/bounded_window_agg_exec.rs
index 9510baab51..4a4c940b22 100644
--- a/datafusion/physical-plan/src/windows/bounded_window_agg_exec.rs
+++ b/datafusion/physical-plan/src/windows/bounded_window_agg_exec.rs
@@ -257,17 +257,11 @@ impl ExecutionPlan for BoundedWindowAggExec {
fn required_input_ordering(&self) -> Vec<Option<LexRequirement>> {
let partition_bys = self.window_expr()[0].partition_by();
let order_keys = self.window_expr()[0].order_by();
- if self.input_order_mode != InputOrderMode::Sorted
- || self.ordered_partition_by_indices.len() >= partition_bys.len()
- {
- let partition_bys = self
- .ordered_partition_by_indices
- .iter()
- .map(|idx| &partition_bys[*idx]);
- vec![calc_requirements(partition_bys, order_keys)]
- } else {
- vec![calc_requirements(partition_bys, order_keys)]
- }
+ let partition_bys = self
+ .ordered_partition_by_indices
+ .iter()
+ .map(|idx| &partition_bys[*idx]);
+ vec![calc_requirements(partition_bys, order_keys)]
}
fn required_input_distribution(&self) -> Vec<Distribution> {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
