jorgecarleitao commented on pull request #11067: URL: https://github.com/apache/arrow/pull/11067#issuecomment-911974343
I skimmed through it and it looks good, thanks a lot for this! What we did in Rust was to use the c data interface that C++ exposes in Python to make calls from within the process, see [here](https://github.com/apache/arrow-rs/tree/master/arrow-pyarrow-integration-testing). I think it would be beneficial to add integration tests against pyarrow. In Rust we found a couple of memory leaks and double frees during development by testing against pyarrow / c++. I would also setup an environment to run those tests against e.g. valgrind, since in FFI is very easy to trigger UB. -- 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
