DSP-15163 support for manager start command with args

jtgrabowski · jtgrabowski · commit 36aa91a25628 · 2018-02-23T13:36:16.000+01:00
we use manager start command with arguments, namely 'dse spark-jobsever
context-per-jvm-managed-start'. This commit reintroduces support for
such commands.
diff --git a/job-server/src/main/scala/spark/jobserver/AkkaClusterSupervisorActor.scala b/job-server/src/main/scala/spark/jobserver/AkkaClusterSupervisorActor.scala
@@ -248,7 +248,7 @@ class AkkaClusterSupervisorActor(daoActor: ActorRef, dataManagerActor: ActorRef)
     }
 
     val contextLogger = LoggerFactory.getLogger("manager_start")
-    val process = Process(managerStartCommand, managerArgs)
+    val process = Process(managerStartCommand.split(" ") ++ managerArgs)
     process.run(ProcessLogger(out => contextLogger.info(out), err => contextLogger.warn(err)))
 
     contextInitInfos(contextActorName) = (mergedContextConfig, isAdHoc, successFunc, failureFunc)

Original file line number	Diff line number	Diff line change
`@@ -248,7 +248,7 @@ class AkkaClusterSupervisorActor(daoActor: ActorRef, dataManagerActor: ActorRef)`
`248`	`248`	`}`
`249`	`249`
`250`	`250`	`val contextLogger = LoggerFactory.getLogger("manager_start")`
`251`		`- val process = Process(managerStartCommand, managerArgs)`
	`251`	`+ val process = Process(managerStartCommand.split(" ") ++ managerArgs)`
`252`	`252`	`process.run(ProcessLogger(out => contextLogger.info(out), err => contextLogger.warn(err)))`
`253`	`253`
`254`	`254`	`contextInitInfos(contextActorName) = (mergedContextConfig, isAdHoc, successFunc, failureFunc)`