Skip to content

Commit

Permalink
Revert "Move RBackend to member variable" (apache#345)
Browse files Browse the repository at this point in the history
  • Loading branch information
pwoody authored Mar 30, 2018
1 parent bb10a57 commit 1010acc
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions core/src/main/scala/org/apache/spark/deploy/RRunner.scala
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,6 @@ import org.apache.spark.util.RedirectThread
* subprocess and then has it connect back to the JVM to access system properties etc.
*/
object RRunner extends CondaRunner with Logging {
val sparkRBackend = new RBackend()

override def run(args: Array[String], maybeConda: Option[CondaEnvironment]): Unit = {
val rFile = PythonRunner.formatPath(args(0))

Expand Down Expand Up @@ -86,6 +84,7 @@ object RRunner extends CondaRunner with Logging {

// Launch a SparkR backend server for the R process to connect to; this will let it see our
// Java system properties etc.
val sparkRBackend = new RBackend()
@volatile var sparkRBackendPort = 0
val initialized = new Semaphore(0)
val sparkRBackendThread = new Thread("SparkR backend") {
Expand Down

0 comments on commit 1010acc

Please sign in to comment.