flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From andralungu <...@git.apache.org>
Subject [GitHub] flink pull request: [FLINK-2661] Added Node Splitting Methods
Date Mon, 14 Sep 2015 16:20:25 GMT
Github user andralungu commented on the pull request:

    Hi @vasia ,
    There was no clear rejection in the discussion, so I thought I'd try my luck :)
    This technique could be enabled by using a flag only if we introduce a node split version
of all the library methods. Otherwise, for newly written code, users would still have to pass
a combine function according to their algorithm. They'd also have to pass the threshold that
differentiates high-degree vertices from low-degree vertices. What is more, they still need
to split and aggregate before and after each step.
    These are aspects that cannot be guessed and that highly depend on the algorithm. 
    Also, what I had in mind regarding this PR was that we would discuss and  try to improve
the code a bit, rather than immediately discarding it :(

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.

View raw message