Github user rmetzger commented on the pull request:

    https://github.com/apache/flink/pull/664#issuecomment-101303549
  
    Yes. But my PR will provide two standard implementations (ParameterTool, 
Configuration), so only if users want more features they need to implement the 
config.
    Users have to cast it in their function, yes.
    
    @hsaputra: I renamed the class to `ParameterTool`.
    
    ----------------------------------
    
    I think the pull request is in a mergeable state now. I didn't change the 
examples yet. Thats going to be a big change which I would like to do 
separately, once this is merged.
    
    Please review the PR ...


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to