Skip to content

Commit 8658ef1

Browse files
committed
fixes
Signed-off-by: Robert Kruszewski <github@robertk.io>
1 parent 8949240 commit 8658ef1

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

vortex-array/src/aggregate_fn/fns/sum/decimal.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
// SPDX-License-Identifier: Apache-2.0
22
// SPDX-FileCopyrightText: Copyright the Vortex contributors
33

4-
use crate::dtype::DecimalDType;
54
use crate::dtype::DecimalType;
65
use itertools::Itertools;
76
use num_traits::AsPrimitive;
@@ -21,6 +20,7 @@ use crate::scalar::DecimalValue;
2120

2221
/// Accumulate a decimal array into the sum state.
2322
/// Returns Ok(true) if saturated (overflow), Ok(false) if not.
23+
#[allow(clippy::cognitive_complexity)]
2424
pub(super) fn accumulate_decimal(inner: &mut SumState, d: &DecimalArray) -> VortexResult<bool> {
2525
let SumState::Decimal { value, dtype } = inner else {
2626
vortex_panic!("expected decimal sum state for decimal input");
@@ -48,7 +48,7 @@ pub(super) fn accumulate_decimal(inner: &mut SumState, d: &DecimalArray) -> Vort
4848
};
4949

5050
let decimal_sum = sum
51-
.map(|v| DecimalValue::from(v))
51+
.map(DecimalValue::from)
5252
// We have to make sure that the decimal value fits the precision of the decimal dtype.
5353
.filter(|v| v.fits_in_precision(*dtype));
5454

0 commit comments

Comments
 (0)