Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10918
---
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 ena
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-184128485
Merged to master
---
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 featu
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-184034314
@srowen okay, all the tests passed, so plz check 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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-184016930
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-184016929
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-184016850
**[Test build #51281 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51281/consoleFull)**
for PR 10918 at commit
[`7d1452d`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183999101
**[Test build #51281 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51281/consoleFull)**
for PR 10918 at commit
[`7d1452d`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183995680
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183995679
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183995661
**[Test build #51280 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51280/consoleFull)**
for PR 10918 at commit
[`24b7580`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183993883
**[Test build #51280 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51280/consoleFull)**
for PR 10918 at commit
[`24b7580`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183989305
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183989297
**[Test build #51279 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51279/consoleFull)**
for PR 10918 at commit
[`a1609ed`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183989307
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183983217
**[Test build #51279 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51279/consoleFull)**
for PR 10918 at commit
[`a1609ed`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183923761
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183923760
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183923705
**[Test build #51270 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51270/consoleFull)**
for PR 10918 at commit
[`8cce85f`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183906582
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183906581
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183906571
**[Test build #51271 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51271/consoleFull)**
for PR 10918 at commit
[`0e301dd`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183905273
**[Test build #51271 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51271/consoleFull)**
for PR 10918 at commit
[`0e301dd`](https://gi
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183904126
@srowen okay, I removed `mapReduceTriplets' in this pr.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183902601
**[Test build #51270 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51270/consoleFull)**
for PR 10918 at commit
[`8cce85f`](https://gi
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-183877548
@maropu I'm ready to merge this, but it needs a rebase, and I think this
should be completed by removing `mapReduceTriplets` entirely
---
If your project is set up for
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-181837507
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-181837503
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-181837114
**[Test build #50967 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50967/consoleFull)**
for PR 10918 at commit
[`12d368a`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-181784627
**[Test build #50967 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50967/consoleFull)**
for PR 10918 at commit
[`12d368a`](https://gi
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-181782798
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 hav
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-181047988
**[Test build #50902 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50902/consoleFull)**
for PR 10918 at commit
[`12d368a`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-181048190
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-181048188
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 projec
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-181030073
Ah, we cannot simply remove some tests because these tests target other
tests for graph processing.
So, we need to replace `mapReduceTriplets` with newer `aggregateMe
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-181027796
**[Test build #50902 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50902/consoleFull)**
for PR 10918 at commit
[`12d368a`](https://gi
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-181027031
If the only usage is tests, then the tests should be removed too, if
they're just there to test this deprecated method. If it's found in other
tests, can you migrate tha
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-181026984
The position of the mima entry fixed.
---
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 doe
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-181026937
Welcome back. We cannot remove `mapReduceTripelets` because some tests in
graphx still use it.
So, I'll make another pr to fix the tests, then remove it.
---
If you
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-180817933
I'm back now. I'm almost ready to merge, but I noticed: should this not
also remove `mapReduceTriplets`? Also the MiMa exclusions are correct but seem
to be in the wrong
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-178393878
okay.
---
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 srowen commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-178390582
I'm out this week on vacation but definitely intend to proceed next week. I
was leaving it for a while in case ankur is around
---
If your project is set up for it, you
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-178373579
@srowen @ankurdave almost inactive, so what should I do?
---
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 maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-175927636
@ankurdave @srowen ping
---
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 thi
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-175067309
Okay
---
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 a
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-175049764
Looks OK to me, though I'lll want to leave it for review for a while. CC
@ankurdave if possible
---
If your project is set up for it, you can reply to this email and ha
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-175030021
@srowen Finished.
---
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 feat
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-175007539
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-175007538
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-175007320
**[Test build #50101 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50101/consoleFull)**
for PR 10918 at commit
[`15937eb`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174964404
**[Test build #50101 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50101/consoleFull)**
for PR 10918 at commit
[`15937eb`](https://gi
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174963992
yeah, I'm working on it.
---
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 th
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174963795
@maropu yeah I think you'll need to write MiMa exclusions for the removal,
to confirm they are intended.
---
If your project is set up for it, you can reply to this ema
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174950622
**[Test build #50097 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50097/consoleFull)**
for PR 10918 at commit
[`346ebdc`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174950666
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174950664
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174947074
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174947082
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174945319
**[Test build #50097 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/50097/consoleFull)**
for PR 10918 at commit
[`346ebdc`](https://gi
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174944377
Ah, yes. I removed runSVDPlusPlus in this pr.
---
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 pro
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174944404
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174944407
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/10918#discussion_r50816340
--- Diff: graphx/src/main/scala/org/apache/spark/graphx/Pregel.scala ---
@@ -109,19 +109,17 @@ object Pregel extends Logging {
* @return the resulting
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174939622
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
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10918#discussion_r50815124
--- Diff: graphx/src/main/scala/org/apache/spark/graphx/Pregel.scala ---
@@ -109,19 +109,17 @@ object Pregel extends Logging {
* @return the resulting
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174938623
While we're here we can remove runSVDPlusPlus in SVDPlusPlus?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174860265
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174860277
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user maropu commented on the pull request:
https://github.com/apache/spark/pull/10918#issuecomment-174859672
@srowen This is an activity from the discussion in #4402.
I checked that GraphX has deprecate APIs used only in Pregel and this pr
removes them.
If there aren't a
GitHub user maropu opened a pull request:
https://github.com/apache/spark/pull/10918
[SPARK-12995][GraphX] Remove deprecate APIs from Pregel
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/maropu/spark RemoveDeprecateInPregel
Al
69 matches
Mail list logo