Skip to content

Commit

Permalink
fix iterator
Browse files Browse the repository at this point in the history
  • Loading branch information
domoritz committed Apr 2, 2024
1 parent ee7a4de commit 355b8e7
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion js/src/visitor/iterator.ts
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,8 @@ function vectorIterator<T extends DataType>(vector: Vector<T>): IterableIterator

// Fast case, defer to native iterators if possible
if (vector.nullCount === 0 && vector.stride === 1 && (
(DataType.isTimestamp(type)) && type.unit === TimeUnit.MILLISECOND ||
// Don't defer to native iterator for timestamps since Numbers are expected
// (DataType.isTimestamp(type)) && type.unit === TimeUnit.MILLISECOND ||
(DataType.isInt(type) && type.bitWidth !== 64) ||
(DataType.isTime(type) && type.bitWidth !== 64) ||
(DataType.isFloat(type) && type.precision !== Precision.HALF)
Expand Down

0 comments on commit 355b8e7

Please sign in to comment.