Github user zentol commented on the pull request:

    https://github.com/apache/flink/pull/2042#issuecomment-222208597
  
    no my concern wasn't even remotely addressed.
    The public static String CONFIG_KEY was removed in commit 
5a7f4e3395bfb06da496584be88501c328f6ac1d, violating the interface stability.
    
    In commit 6c079360c60f47a336eda2f8936a6bddfb948058 you add CONFIG_KEY in 
the ExecutionConfig, while the subsequent commit 
b0acd97935cd21843bac3b9b5afa3662b52bb95d removes it again.
    
    so now the public static key is still removed which shouldn't be the case, 
no?


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