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

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

Github user StephanEwen commented on the pull request:

    https://github.com/apache/flink/pull/952#issuecomment-127184585
  
    Looks good, modulo some styles...
    
      - No need to override all "set*" methods, when the internal set method is 
overridden.
      - Final modifiers should be used for new members where possible, 
consistent with the remaining code
      - Let's not add new Scala classes. We decided that in the long run, it 
would be good to have the runtime  pure Java again.
    
    I need this quite soon, so let me take this over...


> Give access to TaskManager config and hostname in the Runtime Environment
> -------------------------------------------------------------------------
>
>                 Key: FLINK-2425
>                 URL: https://issues.apache.org/jira/browse/FLINK-2425
>             Project: Flink
>          Issue Type: Improvement
>          Components: TaskManager
>    Affects Versions: 0.10
>            Reporter: Stephan Ewen
>            Assignee: Sachin Goel
>             Fix For: 0.10
>
>
> The RuntimeEnvironment (that is used by the operators to access the context) 
> should give access to the TaskManager's configuration, to allow to read 
> config values.



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

Reply via email to