Skip to content

Commit 0d7abee

Browse files
committed
fix
Signed-off-by: Joe Isaacs <joe.isaacs@live.co.uk>
1 parent 06ccc3a commit 0d7abee

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

vortex-array/src/compute/between.rs

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ use crate::compute::Kernel;
2828
use crate::compute::Operator;
2929
use crate::compute::Options;
3030
use crate::compute::Output;
31-
use crate::compute::boolean;
31+
use crate::compute::arrow_boolean;
3232
use crate::compute::compare;
3333
use crate::vtable::VTable;
3434

@@ -149,9 +149,10 @@ impl ComputeFnVTable for Between {
149149

150150
// Otherwise, fall back to the default Arrow implementation
151151
// TODO(joe): should we try to canonicalize the array and try between
152-
Ok(boolean(
153-
&compare(lower, array, options.lower_strict.to_operator())?,
154-
&compare(array, upper, options.upper_strict.to_operator())?,
152+
// TODO(joe): move to lazy execute!
153+
Ok(arrow_boolean(
154+
compare(lower, array, options.lower_strict.to_operator())?,
155+
compare(array, upper, options.upper_strict.to_operator())?,
155156
BooleanOperator::AndKleene,
156157
)?
157158
.into())

0 commit comments

Comments
 (0)