[arrow] branch master updated: ARROW-13265: [R] cli valgrind errors in nightlies

[jonkeane]

This is an automated email from the ASF dual-hosted git repository.

jonkeane pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new 9fead42  ARROW-13265: [R] cli valgrind errors in nightlies
9fead42 is described below

commit 9fead42f50e381d5f9b2cb133bd4693064bdb7f8
Author: Jonathan Keane <jke...@gmail.com>
AuthorDate: Thu Jul 8 21:18:20 2021 -0500

    ARROW-13265: [R] cli valgrind errors in nightlies
    
    Closes #10676 from jonkeane/ARROW-13265-cli-valgrind
    
    Authored-by: Jonathan Keane <jke...@gmail.com>
    Signed-off-by: Jonathan Keane <jke...@gmail.com>
---
 ci/scripts/r_valgrind.sh | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/ci/scripts/r_valgrind.sh b/ci/scripts/r_valgrind.sh
index 1db526e..68c8dd5 100755
--- a/ci/scripts/r_valgrind.sh
+++ b/ci/scripts/r_valgrind.sh
@@ -28,6 +28,12 @@ ${R_BIN} CMD INSTALL ${source_dir}
 pushd ${source_dir}/tests
 
 export TEST_R_WITH_ARROW=TRUE
+
+# Set cli to not use a separate thread. This thread isn't explicitly closed,
+# which triggers a valgrind possibly lost error. We can remove this when
+# https://github.com/r-lib/cli/issues/311 is resolved + released on cran.
+export CLI_NO_THREAD=1
+
 # to generate suppression files run:
 # ${R_BIN} --vanilla -d "valgrind --tool=memcheck --leak-check=full 
--track-origins=yes --gen-suppressions=all --log-file=memcheck.log" -f 
testtthat.supp
 ${R_BIN} --vanilla -d "valgrind --tool=memcheck --leak-check=full 
--track-origins=yes --suppressions=/${1}/ci/etc/valgrind-cran.supp" -f 
testthat.R |& tee testthat.out