alamb commented on PR #7088:
URL:
https://github.com/apache/arrow-datafusion/pull/7088#issuecomment-1657109615
BTW I am sorry for my delay -- unfortunately sorting out list / nested types
is not as high a priority for us at InfluxData at this time, even though I
think it is very important for DataFusion overall. Thus I have to work on this
project in my spare time.
I am definitely still planning to work on this PR, but if anyone else has
time and would like to try and take it on that would also be great 🙏
I was thinking about making `UnnestOptions` in `datafusion_common` and then
update the API to use it
```rust
/// <docs about unnest behavior here>
struct UnestOptions {
pub preserve_nulls: ...
}
```
I would suggest adding a new `options: UnnestOptions` field to the existing
Unest LogicalPlan and UnnestExec rather than making entirely new operators
To keep the size of PRs down, I also would personally suggest adding in the
API in one PR but implementing the new `preserve_nulls: true` behavior as a
second PR.
--
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]
