[GitHub] flink issue #4241: [FLINK-7015] [streaming] separate from OperatorConfig fro...

[aljoscha <gi...@git.apache.org>]

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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---