bjchambers commented on code in PR #23775:
URL: https://github.com/apache/datafusion/pull/23775#discussion_r3641021705
##########
datafusion/functions-nested/src/extract.rs:
##########
@@ -970,7 +970,7 @@ where
#[user_doc(
doc_section(label = "Array Functions"),
- description = "Returns the first non-null element in the array.",
+ description = "Returns the first non-null element in the array. Returns
NULL if the array is empty or all elements are NULL.",
Review Comment:
Done. I was thinking it was somewhat worthwhile since it's not necessarily
obvious what "Returns the first non-null element in the array." does if there
are no non-null elements in the array. But, agreed that `NULL` is the most
likely candidate.
--
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]