Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-4950] Delete obsolete mapReduceTripelets used in Pregel #3782

Closed
wants to merge 2 commits into from

Conversation

maropu
Copy link
Member

@maropu maropu commented Dec 24, 2014

No description provided.

@maropu maropu changed the title SPARK-4950Delete obsolete mapReduceTripelets used in Pregel [SPARK-4950] Delete obsolete mapReduceTripelets used in Pregel Dec 24, 2014
@maropu
Copy link
Member Author

maropu commented Dec 24, 2014

Any reason not to replace the api along with SPARK-3936?

@ankurdave
Copy link
Contributor

We wanted to retain binary compatibility for the Pregel API, which prevented adding the TripletFields parameter. Instead it might be better to add a second version of the Pregel API with several changes: manually-specified TripletFields, aggregateMessages-style API, and custom vertex activeness (#1217).

@maropu maropu force-pushed the DeleteObsoleteAPIInPregel branch from 265a4cf to b8f12f9 Compare December 26, 2014 01:49
@maropu
Copy link
Member Author

maropu commented Dec 26, 2014

Understood. I got back to the old Pregel API.
And also, I'll check #1217 later :))

@sryza
Copy link
Contributor

sryza commented Mar 11, 2015

@maropu mind including [GRAPHX] in the title so that this can get sorted properly?

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@andrewor14
Copy link
Contributor

@maropu @ankurdave is this patch still relevant? If not, I would prefer to close it.

@ankurdave
Copy link
Contributor

I think we can close this PR.

@maropu maropu closed this Jun 20, 2015
@maropu maropu deleted the DeleteObsoleteAPIInPregel branch July 5, 2017 11:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants