+1 We should merge this ASAP. I don't think we could solve all the problems in one PR and I think David's PR is good enough that we can keep working on this incrementally and in parallel.
Regards, Siyuan On Fri, Jul 8, 2016 at 4:10 PM, David Yan <da...@datatorrent.com> wrote: > Hi all, > > The Windowed Operator PR is ready to be merged. Thank you very much for all > your feedback so far. > > https://github.com/apache/apex-malhar/pull/319 > > Merging this PR will make projects related to the WindowedOperator go on > more easily, which includes High level API, Apache Calcite support, Apex > runner in Beam and Dedup operator. Please speak up now If you think there > are reasons for not merging it. > > Also please note that all the classes and interfaces are marked "Evolving" > so we can always change them later. > > Thanks, > > David >