You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@apex.apache.org by David Yan <da...@datatorrent.com> on 2016/07/08 23:10:19 UTC
Windowed Operator PR
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
Re: Windowed Operator PR
Posted by Bhupesh Chawda <bh...@apache.org>.
+1 for merging it.
~ Bhupesh
On Mon, Jul 11, 2016 at 11:10 AM, Yogi Devendra <
devendra.vyavahare@gmail.com> wrote:
> +1 for merging and incremental additions.
>
> ~ Yogi
>
> On 11 July 2016 at 11:04, Chinmay Kolhatkar <ch...@datatorrent.com>
> wrote:
>
> > +1 on merging.
> >
> > On Mon, Jul 11, 2016 at 11:00 AM, Amol Kekre <am...@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 <si...@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
> > > > >
> > > >
> > >
> >
>
Re: Windowed Operator PR
Posted by Yogi Devendra <de...@gmail.com>.
+1 for merging and incremental additions.
~ Yogi
On 11 July 2016 at 11:04, Chinmay Kolhatkar <ch...@datatorrent.com> wrote:
> +1 on merging.
>
> On Mon, Jul 11, 2016 at 11:00 AM, Amol Kekre <am...@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 <si...@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
> > > >
> > >
> >
>
Re: Windowed Operator PR
Posted by Chinmay Kolhatkar <ch...@datatorrent.com>.
+1 on merging.
On Mon, Jul 11, 2016 at 11:00 AM, Amol Kekre <am...@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 <si...@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
> > >
> >
>
Re: Windowed Operator PR
Posted by Amol Kekre <am...@datatorrent.com>.
+1 on merging it. This is a big feature addition.
Thks
Amol
On Fri, Jul 8, 2016 at 4:17 PM, Siyuan Hua <si...@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
> >
>
Re: Windowed Operator PR
Posted by Siyuan Hua <si...@datatorrent.com>.
+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
>