jorisvandenbossche commented on pull request #11527: URL: https://github.com/apache/arrow/pull/11527#issuecomment-968933995
> Why not, but we don't promise to reflect all C++ APIs in the Cython includes. > > Also, we should certainly not start adding tests for this, IMHO. If we don't want to promise this, and don't want to test this, I am not sure if we should actually add it. Because without any test, we have no way to prevent that in a next PR someone might remove it again as clean-up (since it's not used internally), breaking downstream code. (I know this is the same for existing cython APIs as well, if we would stop using one internally, though) -- 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
