izveigor commented on PR #6936: URL: https://github.com/apache/arrow-datafusion/pull/6936#issuecomment-1658961107
> Thank you for the effort @izveigor -- I don't think we should merge this PR with the regression in field access https://github.com/apache/arrow-datafusion/issues/7147 -- among other things we use this feature in IOx so adding a known regression seems like a step backwards I am fully accept the fact that we should prevent the regression. > I didn't fully understand this implementation -- it seem like picking a single value from a list array can be implemented by calculating the appropriate offsets and then calling [take](https://docs.rs/arrow/latest/arrow/compute/kernels/take/fn.take.html) I plan for examine the difference between `array.slice()` and `take` tomorrow. -- 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]
