Github user fanzhidongyzby commented on the issue:
https://github.com/apache/flink/pull/3802
@greghogan , when would this pr be merged ?
---
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 user fanzhidongyzby commented on the issue:
https://github.com/apache/flink/pull/3802
@greghogan , ok, thanks!
---
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
Github user fanzhidongyzby commented on the issue:
https://github.com/apache/flink/pull/3802
@greghogan , `CirculantGraph` and `EchoGraph` are added to `Runner`, and
graph_generators.md is updated well.
---
If your project is set up for it, you can reply to this email and have your
Github user fanzhidongyzby commented on a diff in the pull request:
https://github.com/apache/flink/pull/3802#discussion_r114466541
--- Diff:
flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/generator/CirculantGraph.java
---
@@ -0,0 +1,145
Github user fanzhidongyzby commented on a diff in the pull request:
https://github.com/apache/flink/pull/3802#discussion_r114466564
--- Diff:
flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/generator/CirculantGraph.java
---
@@ -0,0 +1,161
Github user fanzhidongyzby commented on the issue:
https://github.com/apache/flink/pull/3802
@greghogan , The compiling task runs successfully, please have a look at
the submitted code.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user fanzhidongyzby commented on a diff in the pull request:
https://github.com/apache/flink/pull/3802#discussion_r114159796
--- Diff:
flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/generator/CirculantGraph.java
---
@@ -0,0 +1,145
Github user fanzhidongyzby commented on the issue:
https://github.com/apache/flink/pull/3802
@greghogan , What should I do to process this error? All the modified code
has been pushed, how to trigger the pr to be compiled again?
---
If your project is set up for it, you can reply to
Github user fanzhidongyzby commented on the issue:
https://github.com/apache/flink/pull/3802
@greghogan , the compiling error confused me:
```
[ERROR] Plugin org.apache.maven.plugins:maven-surefire-plugin:2.12.2 or one
of its dependencies could not be resolved: Failed to read
Github user fanzhidongyzby commented on a diff in the pull request:
https://github.com/apache/flink/pull/3802#discussion_r114094440
--- Diff:
flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/generator/CirculantGraph.java
---
@@ -0,0 +1,141
Github user fanzhidongyzby commented on a diff in the pull request:
https://github.com/apache/flink/pull/3802#discussion_r114075640
--- Diff:
flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/generator/CirculantGraph.java
---
@@ -0,0 +1,141
Github user fanzhidongyzby commented on the issue:
https://github.com/apache/flink/pull/3802
Hi, @greghogan , the problems mentioned above has been fixed, but some may
needn't to be fixed, the reason has been replied one by one, please review the
code.
---
If your project is s
Github user fanzhidongyzby commented on a diff in the pull request:
https://github.com/apache/flink/pull/3802#discussion_r114075066
--- Diff:
flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/generator/CirculantGraph.java
---
@@ -0,0 +1,141
Github user fanzhidongyzby commented on a diff in the pull request:
https://github.com/apache/flink/pull/3802#discussion_r114074979
--- Diff:
flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/generator/CirculantGraph.java
---
@@ -0,0 +1,141
Github user fanzhidongyzby commented on a diff in the pull request:
https://github.com/apache/flink/pull/3802#discussion_r114074970
--- Diff:
flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/generator/CirculantGraph.java
---
@@ -0,0 +1,141
Github user fanzhidongyzby commented on a diff in the pull request:
https://github.com/apache/flink/pull/3802#discussion_r114074928
--- Diff:
flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/generator/CirculantGraph.java
---
@@ -0,0 +1,141
Github user fanzhidongyzby commented on a diff in the pull request:
https://github.com/apache/flink/pull/3802#discussion_r114074838
--- Diff:
flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/generator/CirculantGraph.java
---
@@ -0,0 +1,141
Github user fanzhidongyzby commented on a diff in the pull request:
https://github.com/apache/flink/pull/3802#discussion_r114074617
--- Diff:
flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/generator/CirculantGraph.java
---
@@ -0,0 +1,141
Github user fanzhidongyzby commented on the issue:
https://github.com/apache/flink/pull/3788
Hi, @greghogan , this pull request was closed unexpectedly by myself,
please go to new [pull request](https://github.com/apache/flink/pull/3802)
recreated.
---
If your project is set up
Github user fanzhidongyzby commented on the issue:
https://github.com/apache/flink/pull/3802
Hi, @greghogan ,
`CirculantGraph` Generator is implemented ok, and `EvenlyGraph` and
`CompleteGraph` was rewrote and tested ok.
---
If your project is set up for it, you can reply to
Github user fanzhidongyzby commented on the issue:
https://github.com/apache/flink/pull/3802
This pull request is recreated, because [last pull request
](https://github.com/apache/flink/pull/3788) was closed by myself unexpectedly.
Please use this new pull request.
---
If
GitHub user fanzhidongyzby opened a pull request:
https://github.com/apache/flink/pull/3802
Add evenly graph
Thanks for contributing to Apache Flink. Before you open your pull request,
please take the following check list into consideration.
If your changes take all of the
Github user fanzhidongyzby closed the pull request at:
https://github.com/apache/flink/pull/3788
---
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
Github user fanzhidongyzby commented on the issue:
https://github.com/apache/flink/pull/3788
@greghogan , I'm preparing to implement `CirculantGraph `, and rewrite
`EmptyGraph`, `CompleteGraph` and `EvenlyGraph` with `CirculantGraph`, is this
ok ?
---
If your project is set u
Github user fanzhidongyzby commented on a diff in the pull request:
https://github.com/apache/flink/pull/3788#discussion_r113848435
--- Diff:
flink-libraries/flink-gelly/src/main/java/org/apache/flink/graph/generator/EvenlyGraph.java
---
@@ -0,0 +1,145 @@
+/*
+ * Licensed
GitHub user fanzhidongyzby opened a pull request:
https://github.com/apache/flink/pull/3788
[FLINK-6393] [gelly] Add Evenly Graph Generator to Flink Gelly
Thanks for contributing to Apache Flink. Before you open your pull request,
please take the following check list into
26 matches
Mail list logo