I think this offset was intended to handle the buffers for vectors that had been [sliced](https://github.com/apache/arrow/blob/master/js/src/vector.ts#L100), but that function is already slicing all of the vector's child buffers [here](https://github.com/apache/arrow/blob/master/js/src/data.ts#L97), so this is redundant. Note that [`subarray`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/TypedArray/subarray) is zero-copy, it's effectively managing the offset for us.
[ Full content available at: https://github.com/apache/arrow/pull/2638 ] This message was relayed via gitbox.apache.org for [email protected]
