Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/11336
could you update the JIRA and PR to reflect what's being added here?
---
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 olarayej commented on the issue:
https://github.com/apache/spark/pull/11336
@felixcheung @falaki I have addressed all your comments and tests pass now.
Thank you! cc @aloknsingh
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66973/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66973 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66973/consoleFull)**
for PR 11336 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66973 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66973/consoleFull)**
for PR 11336 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66968/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66968 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66968/consoleFull)**
for PR 11336 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66968 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66968/consoleFull)**
for PR 11336 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66921 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66921/consoleFull)**
for PR 11336 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66921/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66921 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66921/consoleFull)**
for PR 11336 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66911/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66911 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66911/consoleFull)**
for PR 11336 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66911 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66911/consoleFull)**
for PR 11336 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66905 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66905/consoleFull)**
for PR 11336 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66905/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66905 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66905/consoleFull)**
for PR 11336 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/11336
@wangmiao1981 These are required for the recently added CRAN checks to
ensure we could release SparkR as a R package.
1) - note in-line - they need to have `#' @rdname`
2) - I'm not sure
Github user olarayej commented on the issue:
https://github.com/apache/spark/pull/11336
@falaki @felixcheung I have addressed all your comments. I'm getting two
documentation warnings which seem to be making the build fail:
1)
```
Undocumented S4 methods:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66844 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66844/consoleFull)**
for PR 11336 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66844/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66844 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66844/consoleFull)**
for PR 11336 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66834 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66834/consoleFull)**
for PR 11336 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66834/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66834 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66834/consoleFull)**
for PR 11336 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66832/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66832 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66832/consoleFull)**
for PR 11336 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66832 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66832/consoleFull)**
for PR 11336 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/11336
I know why tests fail - please see my comment.
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66767/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66767 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66767/consoleFull)**
for PR 11336 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66767 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66767/consoleFull)**
for PR 11336 at commit
Github user olarayej commented on the issue:
https://github.com/apache/spark/pull/11336
@falaki Yeah but those warnings are making the build fail (see below). Is
that okay? Now I see a new"Checks" section. I may be outdated with the
protocols as it's been a while I didn't commit :-).
Github user falaki commented on the issue:
https://github.com/apache/spark/pull/11336
I think some of the old tests still rely on `sparkRSQL.init()`. I believe
the warning is OK.
---
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 issue:
https://github.com/apache/spark/pull/11336
@falaki Thanks for your comments. Yeah, before removing collect/show, I
just wanted to rebase to current upstream master. I'm getting a build error
which is actually a warning, not even an R
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66753 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66753/consoleFull)**
for PR 11336 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/11336
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66753/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/11336
**[Test build #66753 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/66753/consoleFull)**
for PR 11336 at commit
Github user olarayej commented on the issue:
https://github.com/apache/spark/pull/11336
@falaki Sorry, I was out of town. Let me get back to this today. Thank you!
---
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 falaki commented on the issue:
https://github.com/apache/spark/pull/11336
@olarayej any update not his. If you are busy I can start another PR from
yours.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user olarayej commented on the issue:
https://github.com/apache/spark/pull/11336
@falaki Absolutely. Let me do that. Thank you!
---
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
Github user falaki commented on the issue:
https://github.com/apache/spark/pull/11336
@olarayej are you interested in rebasing this PR and limiting its user
impact to just `head` for now? We can continue the discussion on support for
distributed vector (a.k.a. `Column`) in JIRA and
52 matches
Mail list logo
