Skip to content

Commit 58febd5

Browse files
fix: drop removed len argument from ScalarFnArray::try_new call
ScalarFnArray::try_new lost its len parameter in #8378 and now infers the length from the first child, which matches the old behavior of passing a.len(). DCO Remediation Commit for Nemo Yu <zyu379@wisc.edu> I, Nemo Yu <zyu379@wisc.edu>, hereby add my Signed-off-by to this commit: 88cb5a2 I, Nemo Yu <zyu379@wisc.edu>, hereby add my Signed-off-by to this commit: 8789430 I, Nemo Yu <zyu379@wisc.edu>, hereby add my Signed-off-by to this commit: 549f5c4 I, Nemo Yu <zyu379@wisc.edu>, hereby add my Signed-off-by to this commit: ec95875 I, Nemo Yu <zyu379@wisc.edu>, hereby add my Signed-off-by to this commit: 874c171 Signed-off-by: Nemo Yu <zyu379@wisc.edu>
1 parent 874c171 commit 58febd5

1 file changed

Lines changed: 0 additions & 2 deletions

File tree

vortex-geo/src/scalar_fn/distance.rs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,11 +46,9 @@ impl GeoDistance {
4646
/// A lazy `ScalarFnArray` computing the per-row distance between the point columns `a` and
4747
/// `b`; either may be constant. The output length is taken from `a`.
4848
pub fn try_new_array(a: ArrayRef, b: ArrayRef) -> VortexResult<ScalarFnArray> {
49-
let len = a.len();
5049
ScalarFnArray::try_new(
5150
TypedScalarFnInstance::new(GeoDistance, EmptyOptions).erased(),
5251
vec![a, b],
53-
len,
5452
)
5553
}
5654
}

0 commit comments

Comments
 (0)