parthchandra opened a new pull request, #1656:
URL: https://github.com/apache/datafusion-comet/pull/1656

   ## 
   Part of #1542 
   
   ## Rationale for this change
   
   In some cases, the field that is being requested does not exist in the file 
being read. With primitive types, one has to check if the default value is 
defined and if so return the default value (otherwise return null). With 
complex types, we merely return null. Existing code was trying to treat complex 
types the same way as primitive types and failing an internal assertion.
   
   
   ## How are these changes tested?
   
   Addition unit test and Spark SQL tests 
   
   (Note: I also added a comment that was requested in a previous PR, not 
related to this change).


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