matthewmturner commented on pull request #1063:
URL: https://github.com/apache/arrow-datafusion/pull/1063#issuecomment-952015566


   @rdettai @Dandandan thank you for your patience on this.  I've made a number 
of updates - would you be able to check it out?  Regarding the point on taking 
into account more cases - should that be done in a separate PR and done in 
conjunction with updating `take_optimizable_min` and `take_optimizable_max` so 
that these optimizations are all aligned?


-- 
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...@arrow.apache.org

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


Reply via email to