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

Greg Hogan commented on FLINK-4204:
-----------------------------------

Like Steve Jobs' iPods, I think Gelly should ship with charged batteries 
[https://www.ted.com/talks/tony_fadell_the_first_secret_of_design_is_noticing/transcript].
 We should make it as easy as possible for new users (not necessarily 
developers) to run algorithms on data and to perceive the power of Flink.

We can also continue to refactor and condense the drivers to reduce the lines 
of code. I hadn't done this because 1) we're still settling on the standard 
functionality and 2) there is more functionality to be added, such as edge 
weights.

I do question, is TriangleListing a useful standalone algorithm? For counting 
triangles ClusteringCoefficient can be used.

> 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
>
> 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