Merge pull request #233 from redbaron/fail-early-on-missing-dep

shivaram · Davies Liu · commit 8b76e810bfd1 · 2015-04-14T13:05:26.000-07:00
Fail worker early if dependency is missing
diff --git a/R/pkg/inst/worker/worker.R b/R/pkg/inst/worker/worker.R
@@ -37,7 +37,7 @@ serializer <- SparkR:::readString(inputCon)
 # Include packages as required
 packageNames <- unserialize(SparkR:::readRaw(inputCon))
 for (pkg in packageNames) {
-  suppressPackageStartupMessages(require(as.character(pkg), character.only=TRUE))
+  suppressPackageStartupMessages(library(as.character(pkg), character.only=TRUE))
 }
 
 # read function dependencies

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ serializer <- SparkR:::readString(inputCon)`
`37`	`37`	`# Include packages as required`
`38`	`38`	`packageNames <- unserialize(SparkR:::readRaw(inputCon))`
`39`	`39`	`for (pkg in packageNames) {`
`40`		`- suppressPackageStartupMessages(require(as.character(pkg), character.only=TRUE))`
	`40`	`+ suppressPackageStartupMessages(library(as.character(pkg), character.only=TRUE))`
`41`	`41`	`}`
`42`	`42`
`43`	`43`	`# read function dependencies`