Merge pull request #343 from pwendell/build-fix
Fix test breaking downstream builds This wasn't detected in the pull-request-builder because it manually sets SPARK_HOME. I'm going to change that (it should't do this) to make it like the other builds.
This commit is contained in:
commit
e4d6057b66
|
@ -26,7 +26,7 @@ import org.apache.spark.deploy.{ExecutorState, Command, ApplicationDescription}
|
|||
class ExecutorRunnerTest extends FunSuite {
|
||||
test("command includes appId") {
|
||||
def f(s:String) = new File(s)
|
||||
val sparkHome = sys.env.get("SPARK_HOME").orElse(sys.env.get("spark.home")).get
|
||||
val sparkHome = sys.env.get("SPARK_HOME").orElse(sys.props.get("spark.home")).get
|
||||
val appDesc = new ApplicationDescription("app name", 8, 500, Command("foo", Seq(),Map()),
|
||||
sparkHome, "appUiUrl")
|
||||
val appId = "12345-worker321-9876"
|
||||
|
|
Loading…
Reference in a new issue