Github user jaceklaskowski commented on the issue:
https://github.com/apache/spark/pull/14680
Thanks @HyukjinKwon You're helping me a lot! I'll work on the unit test.
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14680
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/14680
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64105/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14680
**[Test build #64105 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64105/consoleFull)**
for PR 14680 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14680
Thanks for bearing with me. That was just my personal opinion. As you
already know, I can't decide what should be added into Spark. BTW, we should
fix
Github user jaceklaskowski commented on the issue:
https://github.com/apache/spark/pull/14680
How about now @HyukjinKwon ? The more I look at it the more I think it
should calculated automatically out of the class name when constructor's
called. It's of little to no value to a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14680
**[Test build #64105 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64105/consoleFull)**
for PR 14680 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14680
@jaceklaskowski It seems the test [here]
Github user jaceklaskowski commented on the issue:
https://github.com/apache/spark/pull/14680
@rxin @HyukjinKwon Mind reviewing it again and letting me know what you
think? I know it's minor but would greatly appreciate having it merged at your
earliest convenience. Thanks.
---
If
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14680
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63935/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14680
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/14680
**[Test build #63935 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63935/consoleFull)**
for PR 14680 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14680
**[Test build #63935 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63935/consoleFull)**
for PR 14680 at commit
13 matches
Mail list logo