Github user rmetzger commented on the pull request:

    https://github.com/apache/flink/pull/1978#issuecomment-219992759
  
    Thank you for working on this! 
    The flink-client is pretty complicated and I like this pull request as it 
is reducing the complexity.
    Maybe (depending on the implications) we can further simplify the code and 
introduce some nice abstractions to make the code more maintainable.
    I'll think about it a bit and get back to the PR once all the issues from 
the first pass have been addressed.


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