GitHub user bdwyer2 opened a pull request:
https://github.com/apache/spark/pull/16247
[MINOR][SparkR] set default spark-warehouse path to tempdir()
## What changes were proposed in this pull request?
Set the default location of `spark.sql.warehouse.dir` to be compliant with
the CRAN policy (https://cran.r-project.org/web/packages/policies.html)
regarding writing files outside of the tmp directory. Previously a folder named
`spark-warehouse` was created in the working directory when `sparkR.session()`
was called.
See SPARK-15799 for discussion.
cc @shivaram
## How was this patch tested?
Ran the following code and verified nothing was created in my working
directory:
```R
sparkR.session(master = "local[*]",
sparkConfig = list(spark.driver.memory = "2g"),
enableHiveSupport = FALSE)
```
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/bdwyer2/spark
default_sparkr_spark_warehouse_fix
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/16247.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #16247
----
commit c855c2ce1239650edbaf86a53372adbfd4b3278b
Author: Brendan Dwyer <brendan.dw...@ibm.com>
Date: 2016-12-10T21:00:45Z
set default location of spark.sql.warehouse.dir
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
