Skip to content

Conversation

@benmccann
Copy link
Contributor

No description provided.

@bhoppi
Copy link
Contributor

bhoppi commented Jul 21, 2016

Thank you, but I prefer a full solution that hides EdgeRDDImpl and VertexRDDImpl at the same time, otherwise keep both of them.
But EdgeRDDImpl is hard to optimize by only using provided EdgeRDD APIs.

@benmccann
Copy link
Contributor Author

Yes, EdgeRDDImpl is harder to hide, which is why I left it. But also because you only need to hide VertexRDDImpl in order to use the upstream graphx implementation, so you will get most of the benefit from just hiding VertexRDDImpl whereas hiding EdgeRDDImpl doesn't accomplish much.

@benmccann benmccann closed this Aug 11, 2018
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.

2 participants