Skip to content

Commit 9356c61

Browse files
committed
Remove batching nulls logic from string_array_to_variant
1 parent 55d3d38 commit 9356c61

1 file changed

Lines changed: 4 additions & 13 deletions

File tree

parquet-variant-compute/src/from_json.rs

Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -26,21 +26,12 @@ use parquet_variant_json::JsonToVariant;
2626
/// Macro to convert string array to variant array
2727
macro_rules! string_array_to_variant {
2828
($input:expr, $array:expr, $builder:expr) => {{
29-
let len = $input.len();
30-
let mut i = 0;
31-
while i < len {
32-
if $input.is_valid(i) {
29+
for i in 0..$input.len() {
30+
if $input.is_null(i) {
31+
$builder.append_null();
32+
} else {
3333
$builder.append_json($array.value(i))?;
34-
i += 1;
35-
continue;
3634
}
37-
38-
let start = i;
39-
i += 1;
40-
while i < len && $input.is_null(i) {
41-
i += 1;
42-
}
43-
$builder.append_nulls(i - start);
4435
}
4536
}};
4637
}

0 commit comments

Comments
 (0)