1fanwang commented on code in PR #25970: URL: https://github.com/apache/datafusion/pull/25970#discussion_r4356643797
########## datafusion/functions-nested/src/concat.rs: ########## @@ -419,11 +476,8 @@ fn append_prepend_return_type( /// Concatenates the list arrays in `args` row-wise. /// -/// `field` is the list field the output must carry. `array_concat` passes `None` -/// because its `return_type` derives a fresh field from the unified element -/// types, which is what deriving the field from the aligned inputs reproduces. -/// `array_append` / `array_prepend` promise their input's field verbatim and so -/// must pass it in explicitly. +/// `field` is the list field the output must carry when it differs from the +/// aligned inputs' first field. Review Comment: Done in https://github.com/apache/datafusion/commit/5b91b532420990dc31d790fe2e301959be2b1f80. The doc now distinguishes the planned inner field used by array_concat from the input field used by array_append and array_prepend. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
