Skip to content

Commit 4ffe891

Browse files
committed
cargo fmt
1 parent b828805 commit 4ffe891

1 file changed

Lines changed: 14 additions & 32 deletions

File tree

src/query/expression/src/kernels/stream_partition.rs

Lines changed: 14 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -556,9 +556,7 @@ fn copy_array<I: Index>(
556556
let start = indices[i].to_usize();
557557
let mut run_len = 1;
558558

559-
while i + run_len < indices.len()
560-
&& indices[i + run_len].to_usize() == start + run_len
561-
{
559+
while i + run_len < indices.len() && indices[i + run_len].to_usize() == start + run_len {
562560
run_len += 1;
563561
}
564562

@@ -575,6 +573,8 @@ fn copy_array<I: Index>(
575573

576574
#[cfg(test)]
577575
mod tests {
576+
use databend_common_column::buffer::Buffer;
577+
578578
use super::*;
579579
use crate::FromData;
580580
use crate::types::ArgType;
@@ -583,8 +583,6 @@ mod tests {
583583
use crate::types::Int32Type;
584584
use crate::types::Int64Type;
585585

586-
use databend_common_column::buffer::Buffer;
587-
588586
fn make_block(values: Vec<i32>) -> DataBlock {
589587
DataBlock::new_from_columns(vec![Int32Type::from_data(values)])
590588
}
@@ -666,7 +664,10 @@ mod tests {
666664
// All should have partition_id 0
667665
assert!(result.iter().all(|(id, _)| *id == 0));
668666
// Verify data integrity
669-
let all_values: Vec<i32> = result.iter().flat_map(|(_, b)| collect_column_values(b)).collect();
667+
let all_values: Vec<i32> = result
668+
.iter()
669+
.flat_map(|(_, b)| collect_column_values(b))
670+
.collect();
670671
assert_eq!(all_values, (0..25).collect::<Vec<i32>>());
671672
}
672673

@@ -763,12 +764,7 @@ mod tests {
763764
#[test]
764765
fn test_copy_array_single_partition() {
765766
// All rows go to partition 0, indices are consecutive
766-
let array_col = build_array_column(vec![
767-
vec![1, 2],
768-
vec![3],
769-
vec![4, 5, 6],
770-
vec![7, 8],
771-
]);
767+
let array_col = build_array_column(vec![vec![1, 2], vec![3], vec![4, 5, 6], vec![7, 8]]);
772768
let block = DataBlock::new_from_columns(vec![array_col]);
773769

774770
let mut stream = BlockPartitionStream::create(0, 1, 1);
@@ -784,9 +780,9 @@ mod tests {
784780
fn test_copy_array_multi_partition() {
785781
// Scatter rows across 2 partitions
786782
let array_col = build_array_column(vec![
787-
vec![10, 20], // row 0 -> partition 0
788-
vec![30], // row 1 -> partition 1
789-
vec![40, 50], // row 2 -> partition 0
783+
vec![10, 20], // row 0 -> partition 0
784+
vec![30], // row 1 -> partition 1
785+
vec![40, 50], // row 2 -> partition 0
790786
vec![60, 70, 80], // row 3 -> partition 1
791787
]);
792788
let block = DataBlock::new_from_columns(vec![array_col]);
@@ -810,12 +806,7 @@ mod tests {
810806
#[test]
811807
fn test_copy_array_consecutive_indices() {
812808
// Indices [0,1,2,3] are fully consecutive — exercises the batch path
813-
let array_col = build_array_column(vec![
814-
vec![1],
815-
vec![2, 3],
816-
vec![4, 5, 6],
817-
vec![7],
818-
]);
809+
let array_col = build_array_column(vec![vec![1], vec![2, 3], vec![4, 5, 6], vec![7]]);
819810
let block = DataBlock::new_from_columns(vec![array_col]);
820811

821812
let mut stream = BlockPartitionStream::create(0, 1, 1);
@@ -830,11 +821,7 @@ mod tests {
830821
#[test]
831822
fn test_copy_array_non_consecutive_indices() {
832823
// Indices [0, 2] are non-consecutive — exercises the single-element fallback
833-
let array_col = build_array_column(vec![
834-
vec![10],
835-
vec![20, 30],
836-
vec![40, 50],
837-
]);
824+
let array_col = build_array_column(vec![vec![10], vec![20, 30], vec![40, 50]]);
838825
let block = DataBlock::new_from_columns(vec![array_col]);
839826

840827
let mut stream = BlockPartitionStream::create(0, 1, 2);
@@ -881,12 +868,7 @@ mod tests {
881868
#[test]
882869
fn test_copy_array_empty_inner_arrays() {
883870
// Some inner arrays are empty
884-
let array_col = build_array_column(vec![
885-
vec![],
886-
vec![1, 2],
887-
vec![],
888-
vec![3],
889-
]);
871+
let array_col = build_array_column(vec![vec![], vec![1, 2], vec![], vec![3]]);
890872
let block = DataBlock::new_from_columns(vec![array_col]);
891873

892874
let mut stream = BlockPartitionStream::create(0, 1, 1);

0 commit comments

Comments
 (0)