Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/19657#discussion_r149014391
--- Diff: R/pkg/tests/run-all.R ---
@@ -60,3 +60,22 @@ if (identical(Sys.getenv("NOT_CRAN"), "true")) {
NULL,
"summary")
}
+
+# clean up if Spark was downloaded for the test run
+# get0 not supported before R 3.2.0
+sparkDownloaded <- mget(".sparkDownloaded"[1L],
+ envir = SparkR:::.sparkREnv,
+ inherits = TRUE,
+ ifnotfound = list(FALSE))[[1L]]
+if (sparkDownloaded) {
+ unlink(sparkDownloadedDir, recursive = TRUE, force = TRUE)
+
+ # .cache/spark, or on Windows, LOCALAPPDATA\Apache\Spark\Cache (there
are 3 levels)
+ parentDir <- SparkR:::sparkCachePath()
+ dirs <- list(parentDir, dirname(parentDir), dirname(dirname(parentDir)))
+ lapply(dirs, function(d) {
+ if (length(list.files(d, all.files = TRUE, include.dirs = TRUE, no.. =
TRUE)) == 0) {
--- End diff --
yes, it would, as commented above
https://github.com/apache/spark/pull/19657#issuecomment-341880175
problem is we have no idea whether the vignettes build is going to happen
or not (it could easily be disabled via commandline)
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
