Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename functions-array to functions-nested to collect all nested-type functions #11598

Closed
goldmedal opened this issue Jul 22, 2024 · 2 comments · Fixed by #11602
Closed

Rename functions-array to functions-nested to collect all nested-type functions #11598

goldmedal opened this issue Jul 22, 2024 · 2 comments · Fixed by #11602
Assignees
Labels
enhancement New feature or request

Comments

@goldmedal
Copy link
Contributor

Is your feature request related to a problem or challenge?

Since many implementations of map UDF rely on make_array, we plan to create another crate to collect all the functions for nested types (e.g., array, struct, or map), as discussed in #11572 (comment) and #11546 (comment).

Describe the solution you'd like

Technologically, I plan to rename functions-array that includes the array and map functions currently to functions-nested, then move struct functions to functions-nested.
The array_expressions feature will also be renamed to nested-expressions.

Describe alternatives you've considered

No response

Additional context

No response

@goldmedal goldmedal added the enhancement New feature or request label Jul 22, 2024
@goldmedal
Copy link
Contributor Author

take

@goldmedal goldmedal changed the title Rename functions-arrays to functions-nested and collect all the nested-type function Rename functions-array to functions-nested to collect all nested-type functions Jul 22, 2024
@goldmedal
Copy link
Contributor Author

Let's move the struct functions to functions-nested by another PR.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant