sohailshk commented on issue #27276: URL: https://github.com/apache/datafusion/issues/27276#issuecomment-6761345346
Hi @neilconway, the `expr.slt` case "Position 0 with column input errors even for empty/null inputs" came with #21238, and this issue asks which behavior is intended. If NULL is intended, I would like to take this and implement the fix suggested in the issue: return an all-NULL array from the fast path when the string array has no valid row, before the position check, and change that `expr.slt` case to expect NULL (plus the other two queries from the issue as tests). A batch with at least one non-NULL string would still error for position 0. If the error is intended, I will leave it as is. Any objection? -- 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]
