[
https://issues.apache.org/jira/browse/FLINK-2012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14558445#comment-14558445
]
ASF GitHub Bot commented on FLINK-2012:
---------------------------------------
Github user vasia commented on the pull request:
https://github.com/apache/flink/pull/678#issuecomment-105287317
Hey @andralungu,
first of all, you can always discuss implementation details by starting a
thread in the mailing list. This is usually done for bigger changes, so that we
can all agree before we start coding, instead of discovering disagreements
during review. For small changes like this one, you can simply discuss
implementation on the corresponding JIRA if you want or go ahead and implement
the change and wait for the review :)
Second, my suggestions here are merely that: suggestions :) If you feel
that what I'm suggesting doesn't make sense or you have a better idea, please
discuss your view first before you start coding!
Regarding compatibility, if a method is badly designed or can be improved,
now is the time to fix it. Gelly is not even released yet, so -fortunately- we
still have the time to improve things :)
Finally, regarding the `difference` method, I meant a method that computes
the difference of two graphs and returns a new graph containing the vertices
and edges that the input graphs don't have in common. The idea is that when the
user is dealing with lots of vertices / edges (DataSets), they can use `union`
for additions and `difference` for removals. When the vertices / edges to be
added / removed fit in a List, then the user can use the add/remove methods.
Makes sense?
> addVertices, addEdges, removeVertices, removeEdges methods
> ----------------------------------------------------------
>
> Key: FLINK-2012
> URL: https://issues.apache.org/jira/browse/FLINK-2012
> Project: Flink
> Issue Type: New Feature
> Components: Gelly
> Affects Versions: 0.9
> Reporter: Andra Lungu
> Assignee: Andra Lungu
> Priority: Minor
>
> Currently, Gelly only allows the addition/deletion of one vertex/edge at a
> time. If a user would want to add two (or more) vertices, he/she would need
> to add a vertex-> create a new graph; then add another vertex -> another
> graph etc.
> It would be nice to also have addVertices, addEdges, removeVertices,
> removeEdges methods.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
