Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #3376 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3376/consoleFull)**
for PR 15513 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #3376 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3376/consoleFull)**
for PR 15513 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67612/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
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/15513
**[Test build #67612 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67612/consoleFull)**
for PR 15513 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67612 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67612/consoleFull)**
for PR 15513 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
I will take another look closely as suggested and then will let you all
know.
---
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 HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
I just addressed the comments updated the PR description. I will remove
`[WIP]` although I will double-check again tomorrow.
---
If your project is set up for it, you can reply to this email
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67334 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67334/consoleFull)**
for PR 15513 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67333 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67333/consoleFull)**
for PR 15513 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67274 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67274/consoleFull)**
for PR 15513 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #3366 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3366/consoleFull)**
for PR 15513 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
Oh, FWIW, this build seems being failed correctly in some tests. I will fix
them just to prevent misleading.
---
If your project is set up for it, you can reply to this email and have your
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #3366 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3366/consoleFull)**
for PR 15513 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
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/15513
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67204/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67204 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67204/consoleFull)**
for PR 15513 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67204 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67204/consoleFull)**
for PR 15513 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
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/15513
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67199/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67199 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67199/consoleFull)**
for PR 15513 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67199 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67199/consoleFull)**
for PR 15513 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67130/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
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/15513
**[Test build #67130 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67130/consoleFull)**
for PR 15513 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67130 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67130/consoleFull)**
for PR 15513 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
Thank you @rxin, I just updated the PR description. I will leave the usage
already having single-line usage but just will indent the multiple line ones.
Also, I moved the arguments and examples
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/15513
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/15513
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67071/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67071 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67071/consoleFull)**
for PR 15513 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/15513
I find this too verbose for the basic one. When looking at the basic one, I
want an one-liner explanation because it can also show up in the entire list of
UDFs.
I'd put the detailed argument
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/15513
**[Test build #67071 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67071/consoleFull)**
for PR 15513 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/15513
I suppose I don't know the conventions here well, but, the format looks
better in your change, and more params documentation seems helpful.
---
If your project is set up for it, you can reply to
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
@srowen I will definitely double check the changes here before proceeding
further. Thank you for your review before getting this too big. I will proceed
this in 1-2 days. Please let me know if
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/15513
I am sorry for asking repeatedly but this one is slightly different with
the JIRA one.
cc @rxin @srowen Could you please check if this one is preferable? I am
worried to go further
35 matches
Mail list logo