+1 on merging it. This is a big feature addition. Thks Amol
On Fri, Jul 8, 2016 at 4:17 PM, Siyuan Hua <[email protected]> wrote: > +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 <[email protected]> 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 > > >
