cshuo commented on pull request #16669: URL: https://github.com/apache/flink/pull/16669#issuecomment-894096227
@beyond1920 My main concern is that `translateSessionWindowOperator` is actually duplicated and non-trivial code as that in `StreamExecGroupWindowAggregate`, I think we should avoid maintaining same code in two places. For 2, I not quite clear about what you mean by "limit", routine to old window only changes the runtime operator, which does not affect any plan. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
