[
https://issues.apache.org/jira/browse/FLINK-1976?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14535573#comment-14535573
]
ASF GitHub Bot commented on FLINK-1976:
---------------------------------------
Github user andralungu commented on the pull request:
https://github.com/apache/flink/pull/663#issuecomment-100372745
Sure @hsaputra ,
I thought the JIRA issue was descriptive enough. We were missing
ForwardedFields* annotations for the majority of Gelly classes in Graph.java
especially.
This annotation, as I am sure you know, helps the optimizer generate more
efficient execution plans.
Placing it in front of the classes is optional, but I believe it helps.
@vasia and @fhueske also agreed. In fact this was Fabian's observation when
surfing the Gelly code. Thanks again, Fabian! :)
I hope things are clearer now, @hsaputra. Let me know if you have further
questions :)
> Add ForwardedFields* hints for the optimizer
> ---------------------------------------------
>
> Key: FLINK-1976
> URL: https://issues.apache.org/jira/browse/FLINK-1976
> Project: Flink
> Issue Type: Wish
> Components: Gelly
> Affects Versions: 0.9
> Reporter: Andra Lungu
> Assignee: Andra Lungu
>
> Some classes in Graph.java can be improved by adding ForwardedFields*
> annotations. For instance, EmitOneEdgePerNode,
> EmitOneVertexWithEdgeValuePerNode, EmitOneEdgeWithNeighborPerNode,
> ProjectEdgeWithNeighbor, etc.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
