marc-pydantic opened a new pull request, #18301:
URL: https://github.com/apache/datafusion/pull/18301

   ## Which issue does this PR close?
   
   - Closes #18300
   
   ## Rationale for this change
   
   As laid out in the issue, this improves internal checks by testing an 
assumed invariant, instead of silently nulling data on error. The cost is a 
single null check on a column with a number of entries dependent on the number 
of partitions, not the data itself.
   
   ## What changes are included in this PR?
   
   * Adds a null check to the second column of `merge_batch` of both 
`FIRST_VALUE` and `LAST_VALUE`.
   
   ## Are these changes tested?
   
   Tests are included.
   
   ## Are there any user-facing changes?
   
   Hopefully not.
   


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