+1 for merging it. ~ Bhupesh
On Mon, Jul 11, 2016 at 11:10 AM, Yogi Devendra < devendra.vyavah...@gmail.com> wrote: > +1 for merging and incremental additions. > > ~ Yogi > > On 11 July 2016 at 11:04, Chinmay Kolhatkar <chin...@datatorrent.com> > wrote: > > > +1 on merging. > > > > On Mon, Jul 11, 2016 at 11:00 AM, Amol Kekre <a...@datatorrent.com> > wrote: > > > > > +1 on merging it. This is a big feature addition. > > > > > > Thks > > > Amol > > > > > > > > > On Fri, Jul 8, 2016 at 4:17 PM, Siyuan Hua <siy...@datatorrent.com> > > 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 <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 > > > > > > > > > > > > > > >
