You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "paleolimbot (via GitHub)" <gi...@apache.org> on 2023/06/13 14:10:17 UTC
[GitHub] [arrow-nanoarrow] paleolimbot opened a new pull request, #229: chore: Suppress more jemalloc thread local storage possible leaks and add memcheck for R to verification
paleolimbot opened a new pull request, #229:
URL: https://github.com/apache/arrow-nanoarrow/pull/229
As reported in #200, when loading the arrow R package and allocating at least once there can be an additional thread local storage allocation by jemalloc that leaks. I wasn't able to replicate this in Docker (ubuntu:22.04 or debian:testing, which I believe is the platform CRAN uses); however, their appearance shouldn't cause nanoarrow's memcheck to fail (R or otherwise).
In trying to replicate this, I also realized that release verification wasn't running the R tests with valgrind. On CI this isn't possible because we install arrow from RSPM and that build crashes R under valgrind for some reason. The verification job happens on Docker, though, so we can add the check there.
Closes #200.
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-nanoarrow] paleolimbot merged pull request #229: chore: Suppress more jemalloc thread local storage possible leaks and add memcheck for R to verification
Posted by "paleolimbot (via GitHub)" <gi...@apache.org>.
paleolimbot merged PR #229:
URL: https://github.com/apache/arrow-nanoarrow/pull/229
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-nanoarrow] eddelbuettel commented on pull request #229: chore: Suppress more jemalloc thread local storage possible leaks and add memcheck for R to verification
Posted by "eddelbuettel (via GitHub)" <gi...@apache.org>.
eddelbuettel commented on PR #229:
URL: https://github.com/apache/arrow-nanoarrow/pull/229#issuecomment-1589535473
> which I believe is the platform CRAN uses
AFAIK these extra checks are done by BDR on his Fedora setup. He has text files with config details on his web site but not container, sadly. That said, I tend to mostly be on Ubuntu and running `valgrind` there usually replicates what he finds.
Thanks for taking of this and closing #200. Much appreciated.
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org