alamb commented on PR #6616: URL: https://github.com/apache/arrow-rs/pull/6616#issuecomment-2433522807
> Consider the semantics of take kernel, its output array should not reuse input array. The current behavior looks incorrect. I don't fully understand this assertion. Is the idea that since the [docs](https://docs.rs/arrow/latest/arrow/compute/kernels/take/fn.take.html) say > Take elements by index from [Array](https://docs.rs/arrow/latest/arrow/array/trait.Array.html), creating a new [Array](https://docs.rs/arrow/latest/arrow/array/trait.Array.html) from those indexes. That implies the output array should be entirely new (aka not share anything with the source arrays)? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
