Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16512
It would definitely be considered a new API, though I agree with you that
it's probably safe. That said, I'm not a fan of such changes in patch versions
unless they really are necessary.
---
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/16512
At least from the R code perspective this is source code compatible with
existing 2.1, as adding an optional parameter at the end should not break any
existing code. Also I am not sure I would
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/16512
@shivaram No need to revert this, but why was this merged into 2.1? It's a
new API, and we generally don't backport new APIs.
---
If your project is set up for it, you can reply to this email
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/16512
Merging this to master, branch-2.1
---
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/16512
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/16512
**[Test build #71335 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71335/testReport)**
for PR 16512 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16512
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71335/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16512
**[Test build #71335 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71335/testReport)**
for PR 16512 at commit
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/16512
LGTM. Thanks @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 this feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16512
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71304/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16512
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/16512
**[Test build #71304 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71304/testReport)**
for PR 16512 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16512
**[Test build #71304 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71304/testReport)**
for PR 16512 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16512
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/16512
**[Test build #71300 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71300/testReport)**
for PR 16512 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16512
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71300/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16512
**[Test build #71300 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71300/testReport)**
for PR 16512 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16512
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/16512
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71276/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16512
**[Test build #71276 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71276/testReport)**
for PR 16512 at commit
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/16512
we probably need to add the getNumPartition to compliment 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 project
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16512
**[Test build #71276 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71276/testReport)**
for PR 16512 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16512
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/16512
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71253/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16512
**[Test build #71253 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71253/testReport)**
for PR 16512 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16512
**[Test build #71253 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71253/testReport)**
for PR 16512 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16512
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71182/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16512
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/16512
**[Test build #71182 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71182/testReport)**
for PR 16512 at commit
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/16512
Thanks @felixcheung - CC'ing @falaki @zsxwing from the JIRA
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16512
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71061/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16512
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/16512
**[Test build #71061 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71061/testReport)**
for PR 16512 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16512
**[Test build #71061 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71061/testReport)**
for PR 16512 at commit
34 matches
Mail list logo