Github user aljoscha commented on the issue:

    https://github.com/apache/flink/pull/4241
  
    Hi @juliennju,
    thanks for opening this PR and for interest in changing these parts of 
these code.
    
    However, these changes touch somewhat intricate parts of the code where 
some other people are working and where also several people have plans and 
thoughts about how to improve things. Before doing work on these parts I would 
suggest to start a `[DISCUSS]` thread on the dev mailing list to 1) describe 
your motivation for changing these parts, 2) see what other people have already 
thought about this, and 3) to coordinate changes with other people working on 
this.
    
    I'm sorry for the inconvenience this causes you because you already started 
working on code. 
    
    I'm aware of at least @StefanRRichter, @StephanEwen and me as having spend 
some cycles thinking about changes in these parts.
    
    For example, we would definitely need to discuss what constitutes "operator 
config", "task config" and general "stream config" and how we want to shield 
the different components from each other and make them more modular.
    
    What do you think?


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to