{"action":"created","issue":{"url":"https://api.github.com/repos/apache/flink/issues/408","labels_url":"https://api.github.com/repos/apache/flink/issues/408/labels{/name}","comments_url":"https://api.github.com/repos/apache/flink/issues/408/comments","events_url":"https://api.github.com/repos/apache/flink/issues/408/events","html_url":"https://github.com/apache/flink/pull/408","id":57829597,"number":408,"title":"[FLINK-1514][Gelly] Add a Gather-Sum-Apply iteration method","user":{"login":"balidani","id":1805302,"avatar_url":"https://avatars.githubusercontent.com/u/1805302?v=3","gravatar_id":"","url":"https://api.github.com/users/balidani","html_url":"https://github.com/balidani","followers_url":"https://api.github.com/users/balidani/followers","following_url":"https://api.github.com/users/balidani/following{/other_user}","gists_url":"https://api.github.com/users/balidani/gists{/gist_id}","starred_url":"https://api.github.com/users/balidani/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/balidani/subscriptions","organizations_url":"https://api.github.com/users/balidani/orgs","repos_url":"https://api.github.com/users/balidani/repos","events_url":"https://api.github.com/users/balidani/events{/privacy}","received_events_url":"https://api.github.com/users/balidani/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2015-02-16T17:12:18Z","updated_at":"2015-03-08T14:00:12Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/flink/pulls/408","html_url":"https://github.com/apache/flink/pull/408","diff_url":"https://github.com/apache/flink/pull/408.diff","patch_url":"https://github.com/apache/flink/pull/408.patch"},"body":"Hi!\r\n\r\nI implemented the Gather-Sum-Apply iteration method, it is much nicer now than the previous (`flink-graph`) version. I also added a SSSP test case to the already existing graph coloring one."},"comment":{"url":"https://api.github.com/repos/apache/flink/issues/comments/77750264","html_url":"https://github.com/apache/flink/pull/408#issuecomment-77750264","issue_url":"https://api.github.com/repos/apache/flink/issues/408","id":77750264,"user":{"login":"StephanEwen","id":1727146,"avatar_url":"https://avatars.githubusercontent.com/u/1727146?v=3","gravatar_id":"","url":"https://api.github.com/users/StephanEwen","html_url":"https://github.com/StephanEwen","followers_url":"https://api.github.com/users/StephanEwen/followers","following_url":"https://api.github.com/users/StephanEwen/following{/other_user}","gists_url":"https://api.github.com/users/StephanEwen/gists{/gist_id}","starred_url":"https://api.github.com/users/StephanEwen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/StephanEwen/subscriptions","organizations_url":"https://api.github.com/users/StephanEwen/orgs","repos_url":"https://api.github.com/users/StephanEwen/repos","events_url":"https://api.github.com/users/StephanEwen/events{/privacy}","received_events_url":"https://api.github.com/users/StephanEwen/received_events","type":"User","site_admin":false},"created_at":"2015-03-08T14:00:12Z","updated_at":"2015-03-08T14:00:12Z","body":"I am just dropping some more thoughts here ;-)\r\n\r\nThe interface seems a bit confusing. Right now, one has to first create the iteration from the graph and then tell the graph to run it:\r\n``` java\r\nGatherSumApplyIteration iteration = graph.createGatherSumApplyIteration(gather, sum, apply);\r\nGraph result = graph.runGatherSumApplyIteration(iteration);\r\n```\r\n\r\nWhat was the reason to not go simply for \r\n``` java \r\nGraph result = graph.runGatherSumApplyIteration(gather, sum, apply);\r\n```\r\n\r\nIf the reason was additional parameterization of the iteration, would it be nicer do go for the following?\r\n``` java\r\nGatherSumApplyIteration iteration = graph.createGatherSumApplyIteration(gather, sum, apply);\r\niteration.configureInSomeWay(...);\r\nGraph result = iteration.result();\r\n```\r\n"}}