flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-1632) Use DataSet's count() and collect() to simplify Gelly methods
Date Sat, 07 Mar 2015 16:29:38 GMT

    [ https://issues.apache.org/jira/browse/FLINK-1632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14351671#comment-14351671

ASF GitHub Bot commented on FLINK-1632:

GitHub user andralungu opened a pull request:


    [FLINK-1632][gelly] Deleted GraphUtils

    Made the Graph methods use DataSet's count() instead of GraphUtils.count. 

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/andralungu/flink count

Alternatively you can review and apply these changes as the patch at:


To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #462
commit 73cde6357b5b1672c2ac17730ef8ff5f96f96936
Author: andralungu <lungu.andra@gmail.com>
Date:   2015-03-07T16:17:28Z

    [FLINK-1632][gelly] Deleted GraphUtils and made Gelly methods use DS.count()


> Use DataSet's count() and collect() to simplify Gelly methods
> -------------------------------------------------------------
>                 Key: FLINK-1632
>                 URL: https://issues.apache.org/jira/browse/FLINK-1632
>             Project: Flink
>          Issue Type: Improvement
>          Components: Gelly
>    Affects Versions: 0.9
>            Reporter: Vasia Kalavri
>            Assignee: Andra Lungu
>            Priority: Minor
>              Labels: easyfix, starter
> The recently introduced count() and collect() methods of DataSet can be used to simplify
several Gelly methods. We can get rid of GraphUtils and also simplify methods which return
DataSets with single values, such as numberOfVertices() and isWeaklyConnected().

This message was sent by Atlassian JIRA

View raw message