GitHub user liancheng opened a pull request:
https://github.com/apache/spark/pull/5010
[SPARK-6285] [SQL] Removes unused ParquetTestData and duplicated
TestGroupWriteSupport
All the contents in this file are not referenced anywhere and should have
been removed in #4116 when I tried
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5010#issuecomment-7889
[Test build #28559 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28559/consoleFull)
for PR 5010 at commit
[`06ed057`](https://githu
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/5010#issuecomment-78907978
LGTM, go for it if Jenkins succeeds.
---
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 n
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5010#issuecomment-78913183
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5010#issuecomment-78913165
[Test build #28559 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28559/consoleFull)
for PR 5010 at commit
[`06ed057`](https://gith
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/5010#issuecomment-78966186
jenkins, retest 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 t
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5010#issuecomment-78967889
[Test build #28563 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28563/consoleFull)
for PR 5010 at commit
[`06ed057`](https://githu
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5010#issuecomment-7961
[Test build #28563 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28563/consoleFull)
for PR 5010 at commit
[`06ed057`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5010#issuecomment-7981
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/5010#issuecomment-79517376
@srowen Thanks for the review! Gonna merge this into master and branch-1.3
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/5010
---
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 so, or if the feature is enab
Github user liancheng commented on the pull request:
https://github.com/apache/spark/pull/5010#issuecomment-79519949
Actually I'm not quite sure whether this should backported to branch-1.3 at
this moment, merged to master only for safe. @pwendell?
---
If your project is set up for i
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/5010#issuecomment-79534235
This is more of a cleanup and improvement than bug fix. Although it can
affect the build in some contexts, it does not affect the build of reference.
Generally I think of
13 matches
Mail list logo