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

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

Github user StephanEwen commented on the pull request:

    https://github.com/apache/flink/pull/1004#issuecomment-131148744
  
    Wow, looks like a good piece of work. Nicely with tests and everything.
    Build also passes, style looks good.
    
    +1 to merge this from my side. I'd like to wait for a day or two to get a 
comment from one of the Gelly people (Vasia or Andra).
    
    One thing, though: In the Batch and streaming APIs, we added a 
"completeness check" to make sure that methods added to the Java APIs are also 
present in the Scala APIs. Would that be a good thing to add here as well?


> Add Gelly Scala API
> -------------------
>
>                 Key: FLINK-1962
>                 URL: https://issues.apache.org/jira/browse/FLINK-1962
>             Project: Flink
>          Issue Type: Task
>          Components: Gelly, Scala API
>    Affects Versions: 0.9
>            Reporter: Vasia Kalavri
>            Assignee: PJ Van Aeken
>




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

Reply via email to