assignUser commented on code in PR #12822:
URL: https://github.com/apache/arrow/pull/12822#discussion_r849243297
##########
ci/scripts/r_test.sh:
##########
@@ -90,11 +90,32 @@ SCRIPT="as_cran <-
!identical(tolower(Sys.getenv('NOT_CRAN')), 'true')
message('Running minio for S3 tests (if build supports them)')
minio_dir <- tempfile()
dir.create(minio_dir)
- pid <- sys::exec_background('minio', c('server', minio_dir))
- on.exit(tools::pskill(pid))
+ pid_minio <- sys::exec_background('minio', c('server', minio_dir))
+ on.exit(tools::pskill(pid_minio), add = TRUE)
}
}
+ if(reticulate::py_module_available('pyarrow')){
+ old_pp <- Sys.getenv('PYTHONPATH', unset = NA)
+ Sys.setenv(PYTHONPATH = './inst')
Review Comment:
Prepending the python path does not work with `sys::exec_background` so we
have to manually set and re/unset the python path as not to interfere with any
of the following tests etc..
This would not be necessary with `reticulate::import_from_path`
--
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]
