fallintoplace opened a new pull request, #1135:
URL: https://github.com/apache/arrow-go/pull/1135

   ### Rationale for this change
   
   The filter meta-function assumes the filter datum implements ArrayLikeDatum 
before checking its kind. A record batch or table passed as the filter 
therefore causes a panic instead of the existing unsupported-type error.
   
   ### What changes are included in this PR?
   
   Validate that the filter is array-like before reading its type. Keep scalar 
boolean filters supported and add coverage for a record batch used as the 
filter.
   
   ### Are these changes tested?
   
   - `go test ./arrow/compute`
   
   ### Are there any user-facing changes?
   
   Invalid record or table filters now return an error instead of panicking. 
Valid array, chunked, and scalar boolean filters are unchanged.


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

Reply via email to