Pajaraja opened a new pull request, #51134:
URL: https://github.com/apache/spark/pull/51134

   ### What changes were proposed in this pull request?
   
   Change the output of UnionLoopRef to be nullable by default and use the 
correct output in Local Optimization optimization for small result sets. 
   
   ### Why are the changes needed?
   
   Currently, we mark the nullability of UnionLoopRef with the nullability of 
the anchor. However, the NULL may occur because of the recursive step, leading 
to unexpected behavior (treating NULLs as 0s).
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   New golden file test with failing example.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   No.


-- 
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: reviews-unsubscr...@spark.apache.org

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


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

Reply via email to