Github user tillrohrmann commented on the pull request:
https://github.com/apache/flink/pull/430#issuecomment-76210625
What prevents us from merging this PR? Martin Neumann would like to use
gelly and has asked for documentation.
---
If your project is set up for it, you can reply
Github user vasia commented on the pull request:
https://github.com/apache/flink/pull/430#issuecomment-76223366
Nothing prevent us really, I just need to address your comments! I'll try
to do this later tonight and then I'll merge ;)
---
If your project is set up for it, you can
Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/430#discussion_r25416065
--- Diff: docs/gelly_guide.md ---
@@ -0,0 +1,425 @@
+---
+title: Gelly: Flink Graph API
+---
+!--
+Licensed to the Apache Software
Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/430#discussion_r25416192
--- Diff: docs/gelly_guide.md ---
@@ -0,0 +1,425 @@
+---
+title: Gelly: Flink Graph API
+---
+!--
+Licensed to the Apache Software
Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/430#discussion_r25415850
--- Diff: docs/gelly_guide.md ---
@@ -0,0 +1,425 @@
+---
+title: Gelly: Flink Graph API
+---
+!--
+Licensed to the Apache Software
Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/430#discussion_r25416336
--- Diff: docs/gelly_guide.md ---
@@ -0,0 +1,425 @@
+---
+title: Gelly: Flink Graph API
+---
+!--
+Licensed to the Apache Software
Github user StephanEwen commented on the pull request:
https://github.com/apache/flink/pull/430#issuecomment-75513013
Looks very good, Vasia!
Big +1
---
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
GitHub user vasia opened a pull request:
https://github.com/apache/flink/pull/430
[FLINK-1462] Gelly documentation
Hi,
here's the first draft of the gelly-guide. I have also included the changes
introduced by #402.
Let me know what you think, whether everything is clear,
Github user rmetzger commented on the pull request:
https://github.com/apache/flink/pull/430#issuecomment-75388691
+1 for merging. Very well written.
I now have a rough idea what the Graph API is doing ;)
---
If your project is set up for it, you can reply to this email and have