Github user tony810430 commented on the issue:
https://github.com/apache/flink/pull/3474
Hi @StephanEwen
I think this approach is cleaner than I do.
However, what I am concerned is that `setEnvironment()` method can be
inherited by `AbstractInvokable`'s subclass but constructor can't. It means all
subclasses need to implement this constructor and that is not enforced to do so.
Although we can update all subclasses with an one-argument constructor
which takes `env` only, I think it is not safer than setting `env` by calling a
final method.
What do you think? If there is any mistake in my words, please tell me as
well. Thank you.
---
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.
---
