[GitHub] flink pull request: [FLINK-3846] [gelly] Graph.removeEdges also re...

2016-05-01 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/1948 --- 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] flink pull request: [FLINK-3846] [gelly] Graph.removeEdges also re...

2016-05-01 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/1948#issuecomment-216034401 Merging this one, too! --- 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] flink pull request: [FLINK-3846] [gelly] Graph.removeEdges also re...

2016-04-29 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/1948#issuecomment-215641198 Thanks for the fix! Can you also add a test case that verifies that the bug is fixed? You can simply extend `GraphMutationsITCase`. And +1 to merge. --- If your project

[GitHub] flink pull request: [FLINK-3846] [gelly] Graph.removeEdges also re...

2016-04-28 Thread greghogan
GitHub user greghogan opened a pull request: https://github.com/apache/flink/pull/1948 [FLINK-3846] [gelly] Graph.removeEdges also removes duplicate edges All original edges are now emitted when no matching edge is found in the edge removal set. You can merge this pull request