Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9290
---
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 user shivaram commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152648153
LGTM. Thanks @felixcheung -- Merging this.
---
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
Github user shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/9290#discussion_r43413963
--- Diff: R/pkg/R/sparkR.R ---
@@ -93,7 +93,7 @@ sparkR.stop <- function() {
#' sc <- sparkR.init("local[2]", "SparkR", "/home/spark",
#'
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152287451
**[Test build #44624 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44624/consoleFull)**
for PR 9290 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152286833
Merged build triggered.
---
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
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/9290#discussion_r43430720
--- Diff: R/pkg/R/sparkR.R ---
@@ -93,7 +93,7 @@ sparkR.stop <- function() {
#' sc <- sparkR.init("local[2]", "SparkR", "/home/spark",
#'
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152286858
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152324459
Merged build finished. Test PASSed.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152324463
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152324095
**[Test build #44624 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44624/consoleFull)**
for PR 9290 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151748648
Merged build started.
---
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
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/9290#discussion_r43224710
--- Diff: R/pkg/R/sparkR.R ---
@@ -123,16 +123,24 @@ sparkR.init <- function(
uriSep <- ""
}
+ sparkEnvirMap <-
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151748639
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151753480
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151753361
**[Test build #44509 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44509/consoleFull)**
for PR 9290 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151753478
Merged build finished. Test PASSed.
---
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
Github user sun-rui commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151754033
code looks good.
Note that this works only in SparkR shell mode. This won't work when
executing SparkR scripts, where the JVM launched by spark-submit is the R
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/9290#discussion_r43344210
--- Diff: R/pkg/R/sparkR.R ---
@@ -93,7 +93,7 @@ sparkR.stop <- function() {
#' sc <- sparkR.init("local[2]", "SparkR", "/home/spark",
#'
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/9290#discussion_r43346365
--- Diff: R/pkg/R/sparkR.R ---
@@ -93,7 +93,7 @@ sparkR.stop <- function() {
#' sc <- sparkR.init("local[2]", "SparkR", "/home/spark",
#'
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152029940
**[Test build #44547 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44547/consoleFull)**
for PR 9290 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152036355
**[Test build #44552 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44552/consoleFull)**
for PR 9290 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152038935
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152027236
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152027192
Merged build triggered.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152033222
**[Test build #44547 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44547/consoleFull)**
for PR 9290 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152033478
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152033474
Merged build finished. Test PASSed.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152035196
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152035216
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152038933
Merged build finished. Test PASSed.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-152038850
**[Test build #44552 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44552/consoleFull)**
for PR 9290 at commit
Github user sun-rui commented on a diff in the pull request:
https://github.com/apache/spark/pull/9290#discussion_r43089490
--- Diff: R/pkg/R/sparkR.R ---
@@ -123,16 +123,30 @@ sparkR.init <- function(
uriSep <- ""
}
+ sparkEnvirMap <-
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151383876
Merged build finished. Test PASSed.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151383877
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151383814
**[Test build #44409 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44409/consoleFull)**
for PR 9290 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151719052
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151719064
Merged build started.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151726419
**[Test build #44490 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44490/consoleFull)**
for PR 9290 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151726467
Merged build finished. Test PASSed.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151726470
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151727333
**[Test build #44494 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44494/consoleFull)**
for PR 9290 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151727336
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151727335
Merged build finished. Test FAILed.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151726529
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151726538
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151726940
Merged build triggered.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151727200
**[Test build #44494 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44494/consoleFull)**
for PR 9290 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151720148
**[Test build #44490 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44490/consoleFull)**
for PR 9290 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151726956
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151728226
Merged build finished. Test FAILed.
---
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
GitHub user felixcheung opened a pull request:
https://github.com/apache/spark/pull/9290
[SPARK-11340][SPARKR] Support setting driver properties when starting Spark
from R programmatically or from RStudio
Mapping spark.driver.memory from sparkEnvir to spark-submit commandline
Github user felixcheung commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151377437
Manual testing with:
```
library(SparkR, lib.loc='/opt/spark-1.6.0-bin-hadoop2.6/R/lib')
sc <- sparkR.init(master = "local[*]", sparkEnvir =
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151377420
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151377429
Merged build started.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151381067
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151380796
**[Test build #44405 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44405/consoleFull)**
for PR 9290 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151381066
Merged build finished. Test PASSed.
---
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
Github user felixcheung commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151378899
I checked, the user could also set SPARK_DRIVER_MEMORY before running
`sparkR.init()`
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151382025
**[Test build #44409 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44409/consoleFull)**
for PR 9290 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151378199
**[Test build #44405 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44405/consoleFull)**
for PR 9290 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151379993
Merged build triggered.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9290#issuecomment-151380063
Merged build started.
---
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
62 matches
Mail list logo