Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61881 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61881/consoleFull)**
for PR 14082 at commit
[`e7def7c`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61881 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61881/consoleFull)**
for PR 14082 at commit
[`e7def7c`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61881/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
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 project does not have this feature
e
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/14082
@shivaram @mengxr It would be nice if any of you can help review this one,
thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14082
I'll take a look at this today. Also cc @felixcheung
---
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 t
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14082
looks good, only this minor comment:
https://github.com/apache/spark/pull/14082#discussion_r70001842
and some improvement suggestions to example code.
---
If your project is set up for it,
Github user keypointt commented on the issue:
https://github.com/apache/spark/pull/14082
thanks a lot @felixcheung , working on it
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61946 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61946/consoleFull)**
for PR 14082 at commit
[`1af09f3`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61946 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61946/consoleFull)**
for PR 14082 at commit
[`1af09f3`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61946/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61970 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61970/consoleFull)**
for PR 14082 at commit
[`828b2cf`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61971 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61971/consoleFull)**
for PR 14082 at commit
[`7dca42d`](https://github.com/apache/spark/commit/7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61970 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61970/consoleFull)**
for PR 14082 at commit
[`828b2cf`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61970/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61971 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61971/consoleFull)**
for PR 14082 at commit
[`7dca42d`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61971/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61993 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61993/consoleFull)**
for PR 14082 at commit
[`cd184b3`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61995 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61995/consoleFull)**
for PR 14082 at commit
[`d5b0b7f`](https://github.com/apache/spark/commit/d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61993 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61993/consoleFull)**
for PR 14082 at commit
[`cd184b3`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61993/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #61995 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61995/consoleFull)**
for PR 14082 at commit
[`d5b0b7f`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61995/
Test PASSed.
---
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14082
Thanks @keypointt -- Changes look good to me. @felixcheung any other
comments ?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14082
looks good except for one comment. thanks for putting this together!
---
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 proje
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #62004 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62004/consoleFull)**
for PR 14082 at commit
[`a1eca2b`](https://github.com/apache/spark/commit/a
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14082
LGTM
---
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 f
Github user keypointt commented on the issue:
https://github.com/apache/spark/pull/14082
I just did a quick scan, the no-space style like `session(appName='MyApp',
sparkConfig=list(spark.executor.memory="1g"))` is only found in `example/`
folder (in above commit), and all r files are
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #62005 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62005/consoleFull)**
for PR 14082 at commit
[`7195750`](https://github.com/apache/spark/commit/7
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14082
Thanks @keypointt -- LGTM. Will merge this once Jenkins passes
---
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
Github user keypointt commented on the issue:
https://github.com/apache/spark/pull/14082
Thank you for reviewing :)
---
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 wi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #62005 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62005/consoleFull)**
for PR 14082 at commit
[`7195750`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62005/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14082
**[Test build #62004 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62004/consoleFull)**
for PR 14082 at commit
[`a1eca2b`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62004/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14082
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 project does not have this feature
e
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/14082
Since both @shivaram and @felixcheung signed this off, I'm merging this to
master and branch-2.0.
Thanks @keypointt for working on this and @shivaram and @felixcheung for
the review!
--
43 matches
Mail list logo