izveigor commented on PR #6384: URL: https://github.com/apache/arrow-datafusion/pull/6384#issuecomment-1574235933
Hello, @alamb! I see your and @tustvold comments, thanks for your work! I analyzed all the comments and came to the conclusion that it is better to implement all other changes in subsequent PR, if the current changes do not contain critical errors. (Because it will be easier to analyze changes and implement their) So, I have made a list of issues for possible improvements to arrays: `arrow-rs`: 1) Should some of the features be implemented in `arrow-rs` (for example, `position`)? `arrow-datafusion`: 1) [Important] Implement `unnest` function (it would allow arrays to use aggregate functions `SELECT sum(a) AS total FROM (SELECT unnest(make_array(3, 5, 6) AS a) AS b;` 2) Support NULLS in arrays (not only NullArray) (I think it would be nice to rewrite `make_array` function with using `try_new` method) 3) `array_contains` function (LIKE `array[1, 2, 3] @> array[1, 1, 2, 3]` 4) Write a `Signature` method for list datatypes. 5) Cast between arrays elements. 6) Support empty array? 7) Maybe, refactoring some functions if anyone finds a better solution. 8) `FixedSizeList` to `List` What do you think, @alamb? -- 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]
