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

ASF GitHub Bot commented on FLINK-1632:
---------------------------------------

Github user vasia commented on a diff in the pull request:

    https://github.com/apache/flink/pull/462#discussion_r26009452
  
    --- Diff: 
flink-staging/flink-gelly/src/main/java/org/apache/flink/graph/Graph.java ---
    @@ -919,7 +918,7 @@ public K map(Vertex<K, VV> vertex) {
         *            the maximum number of iterations for the inner delta 
iteration
         * @return true if the graph is weakly connected.
         */
    -   public DataSet<Boolean> isWeaklyConnected(int maxIterations) {
    +   public Boolean isWeaklyConnected(int maxIterations) throws Exception {
    --- End diff --
    
    same :)


> 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
(v6.3.4#6332)

Reply via email to