[GitHub] flink pull request: [FLINK-1885] [gelly] Added bulk execution mode...

2016-01-26 Thread markus-h
Github user markus-h closed the pull request at: https://github.com/apache/flink/pull/598 --- 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-1885] [gelly] Added bulk execution mode...

2016-01-16 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/598#issuecomment-172196328 Hey @markus-h, are there any news regarding this PR? If not, would you mind closing it? Thanks! --- If your project is set up for it, you can reply to this email and

[GitHub] flink pull request: [FLINK-1885] [gelly] Added bulk execution mode...

2015-08-09 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/598#issuecomment-129223234 Hi @markus-h, I'm so sorry it took me so long to look into this.. I agree with Stephan's comment and also it would be great if we could add this option to

[GitHub] flink pull request: [FLINK-1885] [gelly] Added bulk execution mode...

2015-05-12 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/598#issuecomment-101321028 I had a look at this, and it actually looks quite good. The basic idea seems to be that you emit the original vertex if no update happens. It would be nice

[GitHub] flink pull request: [FLINK-1885] [gelly] Added bulk execution mode...

2015-04-17 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/598#issuecomment-93985063 Interesting idea. Are there use cases that require that, or is that basically to allow for an easy comparison of the bulk vs delta performance? --- If your project

[GitHub] flink pull request: [FLINK-1885] [gelly] Added bulk execution mode...

2015-04-17 Thread markus-h
Github user markus-h commented on the pull request: https://github.com/apache/flink/pull/598#issuecomment-93987347 There is no specific usecase, but when you try to process big graphs locally you often run out of memory with delta iterations. But the reason I needed this change is

[GitHub] flink pull request: [FLINK-1885] [gelly] Added bulk execution mode...

2015-04-17 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/598#issuecomment-94002207 Hi @markus-h! I see the point in having a bulk iteration in Gelly, however I'm not sure I would add it as a mode in vertex-centric iteration. VertexCentricIteration

[GitHub] flink pull request: [FLINK-1885] [gelly] Added bulk execution mode...

2015-04-17 Thread markus-h
Github user markus-h commented on the pull request: https://github.com/apache/flink/pull/598#issuecomment-94008273 Hi @vasia, thanks for your comments! I thought about this extension in a different way. Whenever you have a graph that is too big to process it with delta iteration

[GitHub] flink pull request: [FLINK-1885] [gelly] Added bulk execution mode...

2015-04-17 Thread vasia
Github user vasia commented on the pull request: https://github.com/apache/flink/pull/598#issuecomment-94013714 Aha I see! I totally misunderstood your intention :-) I'll take a look as soon as I finish a few more reviews. Thanks! --- If your project is set up for it, you can

[GitHub] flink pull request: [FLINK-1885] [gelly] Added bulk execution mode...

2015-04-14 Thread markus-h
GitHub user markus-h opened a pull request: https://github.com/apache/flink/pull/598 [FLINK-1885] [gelly] Added bulk execution mode to gellys vertex centric iterations See https://issues.apache.org/jira/browse/FLINK-1885 I essentially exchanged the delta iteration with a