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

   ### Rationale for this change
   
   The IsNull helper constructed a unary less comparison, which could panic 
while formatting the expression. IsValid used a function name that is not 
registered. The registered unary functions are is_null and is_not_null.
   
   ### What changes are included in this PR?
   
   Point both helpers at their registered function names and add 
string-formatting coverage.
   
   ### Are these changes tested?
   
   Yes. go test ./arrow/compute
   
   ### Are there any user-facing changes?
   
   IsNull and IsValid now construct usable null-check expressions.


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