paleolimbot commented on PR #446: URL: https://github.com/apache/arrow-nanoarrow/pull/446#issuecomment-2085215526
> is there something release specific about it? Mostly we already had a `TEST_WITH_MEMCHECK` option for release verification (which also runs valgrind over the R and C tests), and so this is the most consistent place to put it (if a little non-standard compared to where another project might put it). It also lets you run this locally with `docker compose run -e TEST_WITH_MEMCHECK=1 verify`. > Should we add Valgrind to normal CI instead of release verification? There are a very small subset of changes to the Python package that could introduce a memory leak (I had to introduce one on purpose to make sure that this worked!), so the weekly verification run is probably sufficient. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
