Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14522
Yeah LGTM. Merging this to master, branch-2.0 -- Thanks @junyangq
---
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 felixcheung commented on the issue:
https://github.com/apache/spark/pull/14522
Since this fixes some CRAN check warnings, let's merge this? @shivaram what
do you think?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14522
LGTM. Minor point about @name - though it will be good to know since we
might tell contributors to always add that
---
If your project is set up for it, you can reply to this email and have
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14522
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63549/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14522
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14522
**[Test build #63549 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63549/consoleFull)**
for PR 14522 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14522
**[Test build #63549 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63549/consoleFull)**
for PR 14522 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14522
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63368/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14522
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14522
**[Test build #63368 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63368/consoleFull)**
for PR 14522 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14522
**[Test build #63368 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63368/consoleFull)**
for PR 14522 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14522
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/14522
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63329/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14522
**[Test build #63329 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63329/consoleFull)**
for PR 14522 at commit
Github user junyangq commented on the issue:
https://github.com/apache/spark/pull/14522
Hi @felixcheung Thanks for reviewing this PR. It happened when I was trying
to fix one of the CRAN warnings (Duplicated \argument entries in documentation
object 'arrange': âxâ) and found it
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14522
**[Test build #63329 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63329/consoleFull)**
for PR 14522 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/14522
Could you elaborate how this help with Spark-16508?
Generally we put functions with the same name even if they have different
parameters in the same doc - but in this case it seems ok since
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14522
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/14522
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63317/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14522
**[Test build #63317 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63317/consoleFull)**
for PR 14522 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14522
**[Test build #63317 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63317/consoleFull)**
for PR 14522 at commit
21 matches
Mail list logo