zanmato1984 commented on issue #45908: URL: https://github.com/apache/arrow/issues/45908#issuecomment-2750804484
> I'm not sure we expose `libarrow_testing` for public consumption. I would be lukewarm about that, since it contains a lot of disparate stuff, some of which we might not want to encourage reliance on. Agreed. Another practical problem about `libarrow_testing` just came to me as well: we have multiple levels of data containers spread in multiple sub-libraries, e.g. `Array`/`RecordBatch`/`Table` in arrow core and `ExecBatch` in compute. We better export `*FromJSON` in the library where `*` belongs to, rather than exporting all `FromJSON` in one place where dependencies would become a mess. -- 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
