jcsherin commented on issue #11433:
URL: https://github.com/apache/datafusion/issues/11433#issuecomment-2229187055

   This is a recent change introduce by #11093. 
   
https://github.com/apache/datafusion/blob/0965455486b7dcbd8c9a5efa8d2370ca5460bb9f/datafusion/functions-aggregate/src/nth_value.rs#L404-L407
   
   In `state_fields` the nullability is set to `false`. But here when 
`Accumulator::evaluate()` is called the nullability of the list is overridden 
to always be `true`. 
   
   It is surprising to me that the schema defined in `state_fields` doesn't 
match the Accumulator and still this works. 
   
   


-- 
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: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to