[arrow-datafusion] branch main updated: Fix GroupByOrderMode documentation (#6168)

[ak...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

akurmustafa pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-datafusion.git


The following commit(s) were added to refs/heads/main by this push:
     new 63ff3f6479 Fix GroupByOrderMode documentation (#6168)
63ff3f6479 is described below

commit 63ff3f6479f244ff43a16e2f82c42156ca40158d
Author: Armin Primadi <ap...@gmail.com>
AuthorDate: Mon May 1 00:09:17 2023 +0700

    Fix GroupByOrderMode documentation (#6168)
---
 datafusion/core/src/physical_plan/aggregates/mod.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/datafusion/core/src/physical_plan/aggregates/mod.rs b/datafusion/core/src/physical_plan/aggregates/mod.rs
index 8539e5fb38..e1f0bfb752 100644
--- a/datafusion/core/src/physical_plan/aggregates/mod.rs
+++ b/datafusion/core/src/physical_plan/aggregates/mod.rs
@@ -80,7 +80,7 @@ pub enum AggregateMode {
 #[derive(Debug, Clone, PartialEq, Eq)]
 pub enum GroupByOrderMode {
     /// Some of the expressions in the GROUP BY clause have an ordering.
-    // For example, if the input is ordered by a, b, c, d and we group by b, a, d;
+    // For example, if the input is ordered by a, b, c and we group by b, a, d;
     // the mode will be `PartiallyOrdered` meaning a subset of group b, a, d
     // defines a preset for the existing ordering, e.g a, b defines a preset.
     PartiallyOrdered,
@@ -510,7 +510,7 @@ impl ExecutionPlan for AggregateExec {
 
     /// Specifies whether this plan generates an infinite stream of records.
     /// If the plan does not support pipelining, but its input(s) are
-    /// infinite, returns an error to indicate this.    
+    /// infinite, returns an error to indicate this.
     fn unbounded_output(&self, children: &[bool]) -> Result<bool> {
         if children[0] {
             if self.aggregation_ordering.is_none() {