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

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


The following commit(s) were added to refs/heads/master by this push:
     new 52d2f7e  Fix typos in doc comments (#945)
52d2f7e is described below

commit 52d2f7e1f2154b5f6e89d4eb9745ffd60e3a1a0c
Author: Andrew Lamb <[email protected]>
AuthorDate: Fri Aug 27 09:46:48 2021 -0400

    Fix typos in doc comments (#945)
---
 datafusion/src/optimizer/utils.rs | 2 +-
 datafusion/tests/sql.rs           | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/datafusion/src/optimizer/utils.rs 
b/datafusion/src/optimizer/utils.rs
index 2f1a081..435daef 100644
--- a/datafusion/src/optimizer/utils.rs
+++ b/datafusion/src/optimizer/utils.rs
@@ -207,7 +207,7 @@ pub fn from_plan(
             })
         }
         LogicalPlan::Explain { .. } => {
-            // Explain shoudl be handled specially in the optimizers;
+            // Explain should be handled specially in the optimizers;
             // If this assert fails it means some optimizer pass is
             // trying to optimize Explain directly
             assert!(
diff --git a/datafusion/tests/sql.rs b/datafusion/tests/sql.rs
index 8aae3d9..adda141 100644
--- a/datafusion/tests/sql.rs
+++ b/datafusion/tests/sql.rs
@@ -3241,7 +3241,7 @@ async fn query_group_on_null_multi_col() -> Result<()> {
     ];
     assert_batches_sorted_eq!(expected, &actual);
 
-    // Also run query with group columns reversed (results shoudl be the same)
+    // Also run query with group columns reversed (results should be the same)
     let sql = "SELECT COUNT(*), c1, c2 FROM test GROUP BY c2, c1";
     let actual = execute_to_batches(&mut ctx, sql).await;
     assert_batches_sorted_eq!(expected, &actual);

Reply via email to