+1
But instead we should add a OpenContext there to keep the signature
stable but still be able to add parameters.
Regards,
Timo
On 21.07.23 12:24, Jing Ge wrote:
+1
On Fri, Jul 21, 2023 at 10:22 AM Yuxin Tan <tanyuxinw...@gmail.com> wrote:
+1
Best,
Yuxin
Xintong Song <tonysong...@gmail.com> 于2023年7月21日周五 12:04写道:
+1
Best,
Xintong
On Fri, Jul 21, 2023 at 10:52 AM Wencong Liu <liuwencle...@163.com>
wrote:
Hi devs,
I would like to start a discussion on FLIP-344: Remove parameter in
RichFunction#open [1].
The open() method in RichFunction requires a Configuration instance as
an
argument,
which is always passed as a new instance without any configuration
parameters in
AbstractUdfStreamOperator#open. Thus, it is unnecessary to include this
parameter
in the open() method.
As such I propose to remove the Configuration field from
RichFunction#open(Configuration parameters).
Looking forward to your feedback.
[1]
https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=263425231
Best regards,
Wencong Liu