SChakravorti21 commented on issue #34535: URL: https://github.com/apache/arrow/issues/34535#issuecomment-1962806793
Hey folks, just wanted to give a heads up that I've created a draft PR at https://github.com/apache/arrow/pull/40226 for the work on this so far. I have a question: I noticed that there is also a [`ChunkedArrayResolver`](https://github.com/apache/arrow/blob/b8fff043c6cb351b1fad87fa0eeaf8dbc550e37c/cpp/src/arrow/compute/kernels/chunked_internal.h#L64) class. This seems to build on top of `ChunkResolver` and provides the added benefit of returning the actual chunk `Array` instead of just its location. I feel like this is convenient for many use-cases and was wondering if we should try to fold `ChunkedArrayResolver`'s behavior into `ChunkResolver` somehow. Do you have any opinions or concerns about this? -- 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]
