Skip to content

[SPARK-18817][SparkR] set default spark-warehouse path to tempdir() #16247

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
bdwyer2 wants to merge 6 commits into from
Closed

[SPARK-18817][SparkR] set default spark-warehouse path to tempdir() #16247

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
c855c2c
set default location of spark.sql.warehouse.dir
bdwyer2 Dec 10, 2016
79caf1c
fixed R style
bdwyer2 Dec 10, 2016
cee3944
added test case
bdwyer2 Dec 12, 2016
17b7bff
added SQL query to test
bdwyer2 Dec 12, 2016
84a110e
attempt to fix failing unit test
bdwyer2 Dec 13, 2016
9142397
remove hive support from unit test
bdwyer2 Dec 13, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
added test case
  • Loading branch information
@bdwyer2
bdwyer2 committed Dec 12, 2016
commit cee39446899387fa40c3e853cd53d3c0d043c4bd
8 changes: 8 additions & 0 deletions R/pkg/inst/tests/testthat/test_sparkR.R
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,3 +44,11 @@ test_that("sparkCheckInstall", {
deployMode <- "client"
expect_error(sparkCheckInstall(sparkHome, master, deployMode))
})

test_that("sparkR.session", {
# nothing should be written outside the tempdir() without explicit user premission
inital_working_directory_files <- list.files()
sparkR.session()
Copy link
Contributor

@shivaram shivaram Dec 12, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we might want to explicitly run some SQL query here and then call sparkR.stop() and then check the files

expect_equal(inital_working_directory_files, list.files())

})