Skip to content

Commit

Permalink
[SPARK-19232][SPARKR] Update Spark distribution download cache locati…
Browse files Browse the repository at this point in the history
…on on Windows

## What changes were proposed in this pull request?

Windows seems to be the only place with appauthor in the path, for which we should say "Apache" (and case sensitive)
Current path of `AppData\Local\spark\spark\Cache` is a bit odd.

## How was this patch tested?

manual.

Author: Felix Cheung <[email protected]>

Closes #16590 from felixcheung/rcachedir.
  • Loading branch information
felixcheung authored and shivaram committed Jan 16, 2017
1 parent 12c8c21 commit a115a54
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions R/pkg/R/install.R
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@
#' \itemize{
#' \item Mac OS X: \file{~/Library/Caches/spark}
#' \item Unix: \env{$XDG_CACHE_HOME} if defined, otherwise \file{~/.cache/spark}
#' \item Windows: \file{\%LOCALAPPDATA\%\\spark\\spark\\Cache}.
#' \item Windows: \file{\%LOCALAPPDATA\%\\Apache\\Spark\\Cache}.
#' }
#' @param overwrite If \code{TRUE}, download and overwrite the existing tar file in localDir
#' and force re-install Spark (in case the local directory or file is corrupted)
Expand Down Expand Up @@ -239,7 +239,7 @@ sparkCachePath <- function() {
"or restart and enter an installation path in localDir.")
stop(msg)
} else {
path <- file.path(winAppPath, "spark", "spark", "Cache")
path <- file.path(winAppPath, "Apache", "Spark", "Cache")
}
} else if (.Platform$OS.type == "unix") {
if (Sys.info()["sysname"] == "Darwin") {
Expand Down

0 comments on commit a115a54

Please sign in to comment.