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

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

GitHub user greghogan opened a pull request:

    https://github.com/apache/flink/pull/2670

    [FLINK-4204] [gelly] Clean up gelly-examples

    Moves drivers into separate package. Adds default main class to print usage 
listing included classes. Includes documentation for running Gelly examples.

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

    $ git pull https://github.com/greghogan/flink 4204_clean_up_gelly_examples

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

    https://github.com/apache/flink/pull/2670.patch

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

    This closes #2670
    
----
commit 642267c70f362ce5414838aaddbed0dcd6b60934
Author: Greg Hogan <[email protected]>
Date:   2016-08-24T15:32:43Z

    [FLINK-4204] [gelly] Clean up gelly-examples
    
    Moves drivers into separate package. Adds default main class to print
    usage listing included classes. Includes documentation for running
    Gelly examples.

----


> Clean up gelly-examples
> -----------------------
>
>                 Key: FLINK-4204
>                 URL: https://issues.apache.org/jira/browse/FLINK-4204
>             Project: Flink
>          Issue Type: Improvement
>          Components: Gelly
>    Affects Versions: 1.1.0
>            Reporter: Vasia Kalavri
>            Assignee: Greg Hogan
>
> The gelly-examples has grown quite big (14 examples) and contains several 
> examples that illustrate the same functionality. Examples should help users 
> understand how to use the API and ideally show how to use 1-2 features.
> Also, it is helpful to state the purpose of each example in the comments.
> We should keep the example set small and move everything that does not fit 
> there to the library.
> I propose to remove the following:
> - ClusteringCoefficient: the functionality already exists as a library method.
> - HITS: the functionality already exists as a library method.
> - JaccardIndex: the functionality already exists as a library method.
> - SingleSourceShortestPaths: the example shows how to use scatter-gather 
> iterations. HITSAlgorithm shows the same feature plus the use of aggregators. 
> I propose we keep this one instead.
> - TriangleListing: the functionality already exists as a library method



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to