buraksenn commented on issue #13037: URL: https://github.com/apache/datafusion/issues/13037#issuecomment-2527809229
> > Should this be done? I saw this #12846. I can delete create_array and record_batch macros to make it use the same as in arrow. Also they are also not identical: > > > > * Datafusion: ($(($name: expr, $type: ident, $values: expr)),*) > > * Arrow: ($(($name: expr, > > t > > y > > p > > e > > : > > i > > d > > e > > n > > t > > , > > [ > > ($values: expr),_])),_) > > > > @timsaucer @alamb ? > > I recommend: > > 1. Switch to use the upstream arrow one > 2. Deprecate (don't remove) the API in datafusion, per https://datafusion.apache.org/library-user-guide/api-health.html Will do as recommended. Thanks @alamb -- 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
