Github user vasia commented on the pull request:
https://github.com/apache/flink/pull/420#issuecomment-129222936
@balidani,
I think it'd be better if you close this PR. I don't think we'll add
another example after #1000 is merged. I can take over and probably reuse some
of your
Github user balidani closed the pull request at:
https://github.com/apache/flink/pull/420
---
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 user balidani commented on the pull request:
https://github.com/apache/flink/pull/420#issuecomment-129223724
@vasia yes, I'm sorry about not finishing it, but I just did not have the
time lately.
Cheers!
---
If your project is set up for it, you can reply to this
Github user vasia commented on the pull request:
https://github.com/apache/flink/pull/420#issuecomment-129226158
Thats' fine :) Thanks for the fast response!
---
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 StephanEwen commented on the pull request:
https://github.com/apache/flink/pull/420#issuecomment-121174894
Any progress on this pull request, or should it be closed?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user balidani commented on the pull request:
https://github.com/apache/flink/pull/420#issuecomment-108284218
Yeah, I should definitely finish this! I'll take a look tonight, sorry
about that :)
---
If your project is set up for it, you can reply to this email and have your
Github user vasia commented on the pull request:
https://github.com/apache/flink/pull/420#issuecomment-108283573
Hey @balidani!
Would you like to finish this up?
It's not really urgent, but it's almost finished and it'd be a pity to
abandon :)
Someone else could also take
Github user vasia commented on the pull request:
https://github.com/apache/flink/pull/420#issuecomment-85487676
Hi @balidani! I think it's still a bit confusing how this example works.
As far as I understand, you expect a directed graph as input, but then you
convert it to an
Github user balidani commented on the pull request:
https://github.com/apache/flink/pull/420#issuecomment-84068746
Hi @vasia! I fixed the algorithm, now it will convert all edges to a pair
of edges and call `distinct` on the edge set. This gives the correct results
now. Thanks!
---
Github user vasia commented on the pull request:
https://github.com/apache/flink/pull/420#issuecomment-77687686
Hi @balidani! Thanks for the example :))
I'm a bit confused about the directed / undirected case. I tried testing
with both a directed and an undirected input and both
Github user vasia commented on a diff in the pull request:
https://github.com/apache/flink/pull/420#discussion_r25995666
--- Diff:
flink-staging/flink-gelly/src/main/java/org/apache/flink/graph/example/LocalClusteringCoefficientExample.java
---
@@ -0,0 +1,205 @@
+/*
+ *
Github user balidani commented on the pull request:
https://github.com/apache/flink/pull/420#issuecomment-76391973
I made the changes @vasia suggested
---
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
Github user vasia commented on the pull request:
https://github.com/apache/flink/pull/420#issuecomment-75586167
Does anyone have an idea about this? Is there a way to pass the
`HashSetK` type?
In any case, even if not, I think it doesn't matter in this case.
I believe we
Github user balidani commented on the pull request:
https://github.com/apache/flink/pull/420#issuecomment-75588338
Hi!
I agree. I will change the example then :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
14 matches
Mail list logo
