Jefffrey opened a new issue, #22507: URL: https://github.com/apache/datafusion/issues/22507
### Describe the bug resize: https://github.com/apache/datafusion/blob/525e01a38c03161bd33104c43afd43c18284aaf6/datafusion/functions-nested/src/resize.rs#L318-L320 - We aren't checking for nulls extract: https://github.com/apache/datafusion/blob/525e01a38c03161bd33104c43afd43c18284aaf6/datafusion/functions-nested/src/extract.rs#L265 - Same replace_n: https://github.com/apache/datafusion/blob/525e01a38c03161bd33104c43afd43c18284aaf6/datafusion/functions-nested/src/replace.rs#L231-L258 - We fixed this in #22387 to equate nulls to be zero, which is what was happening implicitly (since nulls **usually** have 0 value in the underlying array anyway) but we should probably have proper null semantics here (i.e. make the output null) remove_n: https://github.com/apache/datafusion/blob/525e01a38c03161bd33104c43afd43c18284aaf6/datafusion/functions-nested/src/remove.rs#L326-L331 - Keep an eye on #22390 which will handle similarly to above repeat: https://github.com/apache/datafusion/blob/525e01a38c03161bd33104c43afd43c18284aaf6/datafusion/functions-nested/src/repeat.rs#L366-L376 - Similar to above, it explicitly handles as 0 but other systems like DuckDB have proper null semantics here ### To Reproduce _No response_ ### Expected behavior _No response_ ### Additional context _No response_ -- 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]
