Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-146330548
ok merging.
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/8775
---
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
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-146296007
Yes, this was not pushed. I'll push this later if it's still around.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-145988689
[Test build #43279 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43279/console)
for PR 8775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-145989057
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-145989061
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-145990822
@mengxr, looks like the merge commit didn't get pushed (or ASF -> Git
mirroring is lagging)?
---
If your project is set up for it, you can reply to this email and
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-145976691
LGTM. Merged into master. Thanks! There might be a small issue with the
default value. I will test it locally and send follow-up PR if necessary.
---
If your project is
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-145930845
Merged build started.
---
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 vanzin commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-145930335
I'll run the tests once again and push this, since I haven't seen any
feedback on top of the previous PR. retest this please
---
If your project is set up for it, you
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-145930815
Merged build triggered.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-145931732
[Test build #43279 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43279/consoleFull)
for PR 8775 at commit
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-144733008
@JoshRosen #8775
---
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 JoshRosen commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-144576174
Are there plans to revive this PR and attempt another merge?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-143804641
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-143804636
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-143804474
[Test build #43062 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43062/console)
for PR 8775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-143751514
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-143751538
Merged build started.
---
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 vanzin commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-143751385
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 this
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-143753555
[Test build #43062 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43062/consoleFull)
for PR 8775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-142396888
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-142396548
[Test build #42844 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42844/console)
for PR 8775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-142396883
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
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-142351817
any comments on the new approach?
---
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 pull request:
https://github.com/apache/spark/pull/8775#issuecomment-142352557
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-142352527
Merged build triggered.
---
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 vanzin commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-142351787
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 this
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-142353212
[Test build #42844 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42844/consoleFull)
for PR 8775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-140575395
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-140578509
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-140578506
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-140578481
[Test build #42511 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42511/console)
for PR 8775 at commit
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-140574389
Re-posting previous PR with a fix for the issue that caused tests to fail
in other PRs. For the changes since the last PR, see only the second commit.
I'm not
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-140579762
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-140579782
Merged build started.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-140576337
[Test build #42511 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42511/consoleFull)
for PR 8775 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-140580673
[Test build #42514 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42514/consoleFull)
for PR 8775 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-140606243
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-140606242
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-140606177
[Test build #42514 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42514/console)
for PR 8775 at commit
GitHub user vanzin opened a pull request:
https://github.com/apache/spark/pull/8775
[SPARK-10300] [BUILD] [TESTS] Add support for test tags in run-tests.py.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/vanzin/spark
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8775#issuecomment-140575461
Merged build started.
---
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 vanzin commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-140478694
ok, merging.
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/8437
---
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
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-140519235
This seems to have caused PR tests to start failing... I'll try things
locally again and revert the patch if needed.
---
If your project is set up for it, you can reply
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-140520203
Shall we revert the PR first? It is blocking other PRs.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-140521556
Done.
---
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
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-140536012
For the curious, the issue is that the junit test runners (both surefire
and the sbt one) need the tag classes around or they'll throw an error, and
that happens when
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-140193557
No comments? I already got a +1 from Michael, so as far as I can tell, this
is good to go.
---
If your project is set up for it, you can reply to this email and have
Github user pwendell commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-140288092
LGTM - I've modified that script recently enough to be familiar with how it
works. This seems like a good approach and could be useful for us in the future.
---
If
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-138742863
@JoshRosen any comments?
---
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
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-136842836
@JoshRosen should look at 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 does
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135149653
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135149621
Merged build triggered.
---
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 marmbrus commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135143652
SQL changes LGTM. I'll let @pwendell comment on build changes.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135143411
[Test build #41645 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41645/consoleFull)
for PR 8437 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135150079
[Test build #41648 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41648/consoleFull)
for PR 8437 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135172865
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135192321
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135175049
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135175088
Merged build started.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135176979
[Test build #41649 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41649/consoleFull)
for PR 8437 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135192034
[Test build #41645 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41645/console)
for PR 8437 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135192316
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
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135174214
pyspark failures, seem unrelated. in any case... retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135172860
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135172739
[Test build #41642 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41642/console)
for PR 8437 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135193111
[Test build #41648 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41648/console)
for PR 8437 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135193317
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135193314
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
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135193528
Another pyspark failure (that I've seen countless times in other PRs).
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135212316
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 this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135216533
Merged build started.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135217418
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135217178
[Test build #41649 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41649/console)
for PR 8437 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135217416
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135216518
Merged build triggered.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135217619
[Test build #41658 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41658/consoleFull)
for PR 8437 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135213886
Merged build triggered.
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135213945
Merged build started.
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135215475
[Test build #41657 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41657/consoleFull)
for PR 8437 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135278319
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135278321
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135277681
[Test build #41657 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41657/console)
for PR 8437 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135279884
[Test build #41658 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41658/console)
for PR 8437 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135279946
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8437#issuecomment-135279945
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
88 matches
Mail list logo