alamb commented on PR #7088: URL: https://github.com/apache/arrow-datafusion/pull/7088#issuecomment-1661076372
> Sorry for not picking this up. No worries -- sorry for the delay > I'm not sure it makes sense to add the Options as a struct vs just a single preserve_nulls field in the logical/physical plan. Yeah I can see the argument for just a field. The nice thing about the struct was it gave me a place to put the documentation on that could be referenced as well as adding a place that any potential future options could be added > Happy to drive the implementation and include the improvements mentioned https://github.com/apache/arrow-datafusion/issues/6961 That would be great -- 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]
