You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by dh...@apache.org on 2021/12/18 14:41:41 UTC

[arrow-datafusion] branch master updated: fix calculate in many_to_many_hash_partition test. (#1463)

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 35d65fc  fix calculate in many_to_many_hash_partition test. (#1463)
35d65fc is described below

commit 35d65fc37cf2319c4dbec32102adb10502847462
Author: Yang <37...@users.noreply.github.com>
AuthorDate: Sat Dec 18 22:41:36 2021 +0800

    fix calculate in many_to_many_hash_partition test. (#1463)
    
    * fix calculate in many_to_many_hash_partition test.
    
    * fix Clippy Lints
---
 datafusion/src/physical_plan/repartition.rs | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/datafusion/src/physical_plan/repartition.rs b/datafusion/src/physical_plan/repartition.rs
index 55dab6c..3cc7d54 100644
--- a/datafusion/src/physical_plan/repartition.rs
+++ b/datafusion/src/physical_plan/repartition.rs
@@ -576,7 +576,10 @@ mod tests {
         )
         .await?;
 
-        let total_rows: usize = output_partitions.iter().map(|x| x.len()).sum();
+        let total_rows: usize = output_partitions
+            .iter()
+            .map(|x| x.iter().map(|x| x.num_rows()).sum::<usize>())
+            .sum();
 
         assert_eq!(8, output_partitions.len());
         assert_eq!(total_rows, 8 * 50 * 3);