ShashidharM0118 opened a new pull request, #19242:
URL: https://github.com/apache/datafusion/pull/19242

   ## Which issue does this PR close?
   
   - Closes #19158
   
   ## Rationale for this change
   
   The Spark `sha1` UDF was always marked nullable because it relied on the 
default nullability; Spark semantics require the output nullability to follow 
the input (or null scalar) instead.
   
   ## What changes are included in this PR?
   
   - Implement `return_field_from_args` for `SparkSha1` to report `Utf8` with 
nullability derived from argument fields or null scalar args, and guard 
`return_type`.
   - Add a nullability unit test covering non-nullable input, nullable input, 
and null scalar cases.
   
   ## Are these changes tested?
   
   Yes,  
   ## Are there any user-facing changes?
   
   No behavior change beyond correct nullability reporting for `sha1` results.
   


-- 
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]

Reply via email to