Skip to content

Commit

Permalink
Implement maintains_input_order for AggregateExec
Browse files Browse the repository at this point in the history
  • Loading branch information
alihan-synnada committed Dec 24, 2024
1 parent b4b267a commit 49ab893
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions datafusion/physical-plan/src/aggregates/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -792,6 +792,11 @@ impl ExecutionPlan for AggregateExec {
vec![self.required_input_ordering.clone()]
}

fn maintains_input_order(&self) -> Vec<bool> {
// See: InputOrderMode documentation for aggregations
vec![self.input_order_mode != InputOrderMode::Linear]
}

fn children(&self) -> Vec<&Arc<dyn ExecutionPlan>> {
vec![&self.input]
}
Expand Down

0 comments on commit 49ab893

Please sign in to comment.