Hi devs,

I would like to start a discussion on FLIP-343: Remove parameter in 
WindowAssigner#getDefaultTrigger() [1].


The method getDefaultTrigger() in WindowAssigner takes a 
StreamExecutionEnvironment 
parameter, but this parameter is not actually used for any subclasses of 
WindowAssigner. 
Therefore, it is unnecessary to include this parameter.
As such I propose to remove the StreamExecutionEnvironment field from 
WindowAssigner#getDefaultTrigger(StreamExecutionEnvironment env).
Looking forward to your feedback.
[1] https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=263425229
Best regards,


Wencong Liu

Reply via email to