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