Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/9579
---
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 enab
Github user shivaram commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155533653
Thanks @olarayej @sun-rui and @felixcheung. LGTM and merging this
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user olarayej commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155347716
@felixcheung @sun-rui All set. Can we please merge? Thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155342496
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155342487
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155342110
**[Test build #45499 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45499/consoleFull)**
for PR 9579 at commit
[`01366a6`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155330800
**[Test build #45499 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45499/consoleFull)**
for PR 9579 at commit
[`01366a6`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155327387
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155327337
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 h
Github user olarayej commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155326781
I have addressed the docs suggestions. Thank you folks.
---
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 sun-rui commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155266497
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 a
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155261557
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155261555
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155261477
**[Test build #45461 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45461/consoleFull)**
for PR 9579 at commit
[`a031aed`](https://git
Github user felixcheung commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155258819
doc comment, looks good to me otherwise.
@sun-rui @shivaram
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/9579#discussion_r44361878
--- Diff: R/pkg/R/DataFrame.R ---
@@ -2152,3 +2152,47 @@ setMethod("with",
newEnv <- assignNewEnv(data)
eval(substitute(e
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155257549
**[Test build #45461 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45461/consoleFull)**
for PR 9579 at commit
[`a031aed`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155257019
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155256971
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155254690
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155254692
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/4
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155254544
**[Test build #45453 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45453/consoleFull)**
for PR 9579 at commit
[`0eabaf8`](https://git
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/9579#discussion_r44359782
--- Diff: R/pkg/R/schema.R ---
@@ -115,20 +115,7 @@ structField.jobj <- function(x) {
}
checkType <- function(type) {
- primtiveTypes
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155249736
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155249756
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 have
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155249801
**[Test build #45453 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45453/consoleFull)**
for PR 9579 at commit
[`0eabaf8`](https://gith
Github user yu-iskw commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155249581
@olarayej could you please check the style warnings with `./dev/lint-r` on
your local computer before pushing commits. Thanks.
---
If your project is set up for it, you
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155248527
**[Test build #45451 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45451/consoleFull)**
for PR 9579 at commit
[`635bbe7`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155248535
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155248221
**[Test build #45451 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/45451/consoleFull)**
for PR 9579 at commit
[`635bbe7`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155248162
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9579#issuecomment-155248181
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 have
GitHub user olarayej opened a pull request:
https://github.com/apache/spark/pull/9579
[SPARK-10863][SPARKR] Method coltypes() (New version)
This is a follow up on PR #8984, as the corresponding branch for such PR
was damaged.
You can merge this pull request into a Git repository by
33 matches
Mail list logo