[GitHub] flink pull request: [FLINK-1528][Gelly] Added Local Clustering Coe...

[vasia <gi...@git.apache.org>]

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 `HashSet<K>` type?
    
    In any case, even if not, I think it doesn't matter in this case.
    I believe we shouldn't add this as a library method, as it is a quite naive implementation of local clustering coefficient. I'd prefer if we try to keep the library methods as efficient as possible.
    
    However, I would definitely add this as an example, since it very nicely demonstrates how to use neighborhood methods and `joinWithVertices` (which are missing from the other examples).
    So, I would suggest we change this to an example that uses a sample dataset, with e.g. Long ids and also allows file input. What do you think @balidani?


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---