[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2017-07-19 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r128311859 --- Diff: R/pkg/R/backend.R --- @@ -108,13 +108,27 @@ invokeJava <- function(isStatic, objId, methodName, ...) { conn <- get(".sparkRCon",

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2017-07-19 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r128301290 --- Diff: R/pkg/R/backend.R --- @@ -108,13 +108,27 @@ invokeJava <- function(isStatic, objId, methodName, ...) { conn <- get(".sparkRCon",

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2017-07-18 Thread QCTW
Github user QCTW commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r127771891 --- Diff: R/pkg/R/backend.R --- @@ -108,13 +108,27 @@ invokeJava <- function(isStatic, objId, methodName, ...) { conn <- get(".sparkRCon", .sparkREnv)

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/15471 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-26 Thread zsxwing
Github user zsxwing commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r85248929 --- Diff: core/src/main/scala/org/apache/spark/api/r/RBackendHandler.scala --- @@ -83,7 +86,29 @@ private[r] class RBackendHandler(server: RBackend)

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-26 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r85247619 --- Diff: core/src/main/scala/org/apache/spark/api/r/RBackendHandler.scala --- @@ -83,7 +86,29 @@ private[r] class RBackendHandler(server: RBackend)

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-26 Thread falaki
Github user falaki commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r85230441 --- Diff: core/src/main/scala/org/apache/spark/api/r/RBackendHandler.scala --- @@ -83,7 +86,29 @@ private[r] class RBackendHandler(server: RBackend)

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-26 Thread falaki
Github user falaki commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r85229665 --- Diff: core/src/main/scala/org/apache/spark/api/r/RBackend.scala --- @@ -110,6 +115,11 @@ private[spark] object RBackend extends Logging { val

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-24 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r84730391 --- Diff: core/src/main/scala/org/apache/spark/api/r/RRunner.scala --- @@ -333,6 +333,8 @@ private[r] object RRunner { var rCommand =

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-24 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r84113455 --- Diff: R/pkg/inst/worker/daemon.R --- @@ -18,6 +18,7 @@ # Worker daemon rLibDir <- Sys.getenv("SPARKR_RLIBDIR") +connectionTimeout <-

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-24 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r84730154 --- Diff: core/src/main/scala/org/apache/spark/api/r/RBackend.scala --- @@ -110,6 +115,11 @@ private[spark] object RBackend extends Logging { val

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-24 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r84729305 --- Diff: R/pkg/inst/worker/worker.R --- @@ -90,6 +90,7 @@ bootTime <- currentTimeSecs() bootElap <- elapsedSecs() rLibDir <-

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-24 Thread shivaram
Github user shivaram commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r84730780 --- Diff: core/src/main/scala/org/apache/spark/api/r/RBackendHandler.scala --- @@ -83,7 +86,29 @@ private[r] class RBackendHandler(server: RBackend)

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-19 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r84171665 --- Diff: core/src/main/scala/org/apache/spark/api/r/RBackendHandler.scala --- @@ -83,7 +86,29 @@ private[r] class RBackendHandler(server: RBackend)

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-19 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r84171676 --- Diff: R/pkg/R/backend.R --- @@ -108,13 +108,27 @@ invokeJava <- function(isStatic, objId, methodName, ...) { conn <- get(".sparkRCon",

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-19 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r84171310 --- Diff: core/src/main/scala/org/apache/spark/api/r/RBackendHandler.scala --- @@ -83,7 +86,29 @@ private[r] class RBackendHandler(server: RBackend)

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-19 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/15471#discussion_r84172455 --- Diff: core/src/main/scala/org/apache/spark/api/r/RBackendHandler.scala --- @@ -95,9 +120,15 @@ private[r] class RBackendHandler(server: RBackend)

[GitHub] spark pull request #15471: [SPARK-17919] Make timeout to RBackend configurab...

2016-10-13 Thread falaki
GitHub user falaki opened a pull request: https://github.com/apache/spark/pull/15471 [SPARK-17919] Make timeout to RBackend configurable in SparkR ## What changes were proposed in this pull request? This patch makes RBackend connection timeout configurable by user.