alamb commented on issue #1610: URL: https://github.com/apache/arrow-datafusion/issues/1610#issuecomment-1018759711
> Does this make sense? Yes I think that makes sense to me. I think they key outcome of this PR should be that `eq_dyn_scalar` etc are called, rather than any particular macros are removed from DataFusion's code. Overtime we can look to remove / simplify macros as the kernels in arrow get more full fetaures, but it doesn't have to happen now -- 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
