Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/1
LGTM
I'll merge this with master
Thanks!
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #67575 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67575/consoleFull)**
for PR 1 at commit
[`946ee73`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67575/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #67575 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67575/consoleFull)**
for PR 1 at commit
[`946ee73`](https://github.com/apache/spark/commit/9
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/1
I'd say add Since tags wherever applicable in concrete classes. But if
it's an abstract method, we probably should not since they will be incorrect
for new child classes later on.
---
If your p
Github user sethah commented on the issue:
https://github.com/apache/spark/pull/1
@yanboliang What do you think about adding the `@Since` tags?
---
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
Github user zhengruifeng commented on the issue:
https://github.com/apache/spark/pull/1
Looks good to me.
---
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 wishes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #67494 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67494/consoleFull)**
for PR 1 at commit
[`f13f240`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67494/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #67494 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67494/consoleFull)**
for PR 1 at commit
[`f13f240`](https://github.com/apache/spark/commit/f
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/1
jenkins test this please
---
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 wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #67490 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67490/consoleFull)**
for PR 1 at commit
[`f13f240`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67490/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #67490 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67490/consoleFull)**
for PR 1 at commit
[`f13f240`](https://github.com/apache/spark/commit/f
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/1
@srowen Would you mind to have a look when you available? Thanks.
---
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 d
Github user yanboliang commented on the issue:
https://github.com/apache/spark/pull/1
cc @zhengruifeng @srowen
---
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 wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/1
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/67198/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #67198 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67198/consoleFull)**
for PR 1 at commit
[`3169267`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/1
**[Test build #67198 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/67198/consoleFull)**
for PR 1 at commit
[`3169267`](https://github.com/apache/spark/commit/3
23 matches
Mail list logo
