Removing flink-contrib/flink-operator-stats

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

Removing flink-contrib/flink-operator-stats

Ufuk Celebi-2
Hey devs,

I would like to propose the removal of the
flink-contrib/flink-operator-stats module.

It is currently causing some build stability issues
(https://issues.apache.org/jira/browse/FLINK-4833) and there is no
active maintainer for it as far as I can tell.

Are there any objections to this?

I know it always feel bad to remove contributed code, but given the
size and scope of the project I think that it is fair to be
conservative about these issues. I think we always planned the contrib
module to be a staging area where we monitor the usefulness of
non-core additions.

Best,

Ufuk
Reply | Threaded
Open this post in threaded view
|

Re: Removing flink-contrib/flink-operator-stats

Robert Metzger
If there are no users and no contributors of the module, I'm +1 to remove
it.

If we decide to remove it from flink-contrib, but there are some
contributors interested in it, I can offer to assist the contributors to
add the extension to Apache Bahir.

On Wed, Oct 19, 2016 at 2:00 PM, Ufuk Celebi <[hidden email]> wrote:

> Hey devs,
>
> I would like to propose the removal of the
> flink-contrib/flink-operator-stats module.
>
> It is currently causing some build stability issues
> (https://issues.apache.org/jira/browse/FLINK-4833) and there is no
> active maintainer for it as far as I can tell.
>
> Are there any objections to this?
>
> I know it always feel bad to remove contributed code, but given the
> size and scope of the project I think that it is fair to be
> conservative about these issues. I think we always planned the contrib
> module to be a staging area where we monitor the usefulness of
> non-core additions.
>
> Best,
>
> Ufuk
>
mxm
Reply | Threaded
Open this post in threaded view
|

Re: Removing flink-contrib/flink-operator-stats

mxm
+1 for removing it in case it is not widely used. Apache Bahir would
be a more appropriate place for this module then.

-Max


On Wed, Oct 19, 2016 at 3:52 PM, Robert Metzger <[hidden email]> wrote:

> If there are no users and no contributors of the module, I'm +1 to remove
> it.
>
> If we decide to remove it from flink-contrib, but there are some
> contributors interested in it, I can offer to assist the contributors to
> add the extension to Apache Bahir.
>
> On Wed, Oct 19, 2016 at 2:00 PM, Ufuk Celebi <[hidden email]> wrote:
>
>> Hey devs,
>>
>> I would like to propose the removal of the
>> flink-contrib/flink-operator-stats module.
>>
>> It is currently causing some build stability issues
>> (https://issues.apache.org/jira/browse/FLINK-4833) and there is no
>> active maintainer for it as far as I can tell.
>>
>> Are there any objections to this?
>>
>> I know it always feel bad to remove contributed code, but given the
>> size and scope of the project I think that it is fair to be
>> conservative about these issues. I think we always planned the contrib
>> module to be a staging area where we monitor the usefulness of
>> non-core additions.
>>
>> Best,
>>
>> Ufuk
>>
Reply | Threaded
Open this post in threaded view
|

Re: Removing flink-contrib/flink-operator-stats

Greg Hogan
Based on a cursory reading of FLINK-1297 I would lean toward dropping the
code rather than moving to Apache Bahir. This looks to only be appropriate
for batch and this module was not integrated into the runtime.

If there is a way forward to make use this code in core Flink then that
would be even better.

On Wed, Oct 19, 2016 at 9:54 AM, Maximilian Michels <[hidden email]> wrote:

> +1 for removing it in case it is not widely used. Apache Bahir would
> be a more appropriate place for this module then.
>
> -Max
>
>
> On Wed, Oct 19, 2016 at 3:52 PM, Robert Metzger <[hidden email]>
> wrote:
> > If there are no users and no contributors of the module, I'm +1 to remove
> > it.
> >
> > If we decide to remove it from flink-contrib, but there are some
> > contributors interested in it, I can offer to assist the contributors to
> > add the extension to Apache Bahir.
> >
> > On Wed, Oct 19, 2016 at 2:00 PM, Ufuk Celebi <[hidden email]> wrote:
> >
> >> Hey devs,
> >>
> >> I would like to propose the removal of the
> >> flink-contrib/flink-operator-stats module.
> >>
> >> It is currently causing some build stability issues
> >> (https://issues.apache.org/jira/browse/FLINK-4833) and there is no
> >> active maintainer for it as far as I can tell.
> >>
> >> Are there any objections to this?
> >>
> >> I know it always feel bad to remove contributed code, but given the
> >> size and scope of the project I think that it is fair to be
> >> conservative about these issues. I think we always planned the contrib
> >> module to be a staging area where we monitor the usefulness of
> >> non-core additions.
> >>
> >> Best,
> >>
> >> Ufuk
> >>
>
Reply | Threaded
Open this post in threaded view
|

Re: Removing flink-contrib/flink-operator-stats

Stephan Ewen
+1 for removing it

  - It seems quite unstable (is responsible for almost all build failures
right now)
  - It is not integrated with the metric system. Having more metrics is
desirable, but is a separate effort and needs a different approach.

On Wed, Oct 19, 2016 at 4:23 PM, Greg Hogan <[hidden email]> wrote:

> Based on a cursory reading of FLINK-1297 I would lean toward dropping the
> code rather than moving to Apache Bahir. This looks to only be appropriate
> for batch and this module was not integrated into the runtime.
>
> If there is a way forward to make use this code in core Flink then that
> would be even better.
>
> On Wed, Oct 19, 2016 at 9:54 AM, Maximilian Michels <[hidden email]>
> wrote:
>
> > +1 for removing it in case it is not widely used. Apache Bahir would
> > be a more appropriate place for this module then.
> >
> > -Max
> >
> >
> > On Wed, Oct 19, 2016 at 3:52 PM, Robert Metzger <[hidden email]>
> > wrote:
> > > If there are no users and no contributors of the module, I'm +1 to
> remove
> > > it.
> > >
> > > If we decide to remove it from flink-contrib, but there are some
> > > contributors interested in it, I can offer to assist the contributors
> to
> > > add the extension to Apache Bahir.
> > >
> > > On Wed, Oct 19, 2016 at 2:00 PM, Ufuk Celebi <[hidden email]> wrote:
> > >
> > >> Hey devs,
> > >>
> > >> I would like to propose the removal of the
> > >> flink-contrib/flink-operator-stats module.
> > >>
> > >> It is currently causing some build stability issues
> > >> (https://issues.apache.org/jira/browse/FLINK-4833) and there is no
> > >> active maintainer for it as far as I can tell.
> > >>
> > >> Are there any objections to this?
> > >>
> > >> I know it always feel bad to remove contributed code, but given the
> > >> size and scope of the project I think that it is fair to be
> > >> conservative about these issues. I think we always planned the contrib
> > >> module to be a staging area where we monitor the usefulness of
> > >> non-core additions.
> > >>
> > >> Best,
> > >>
> > >> Ufuk
> > >>
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: Removing flink-contrib/flink-operator-stats

Ufuk Celebi-2
I'll go ahead and remove it today.

On Thu, Oct 20, 2016 at 8:21 PM, Stephan Ewen <[hidden email]> wrote:

> +1 for removing it
>
>   - It seems quite unstable (is responsible for almost all build failures
> right now)
>   - It is not integrated with the metric system. Having more metrics is
> desirable, but is a separate effort and needs a different approach.
>
> On Wed, Oct 19, 2016 at 4:23 PM, Greg Hogan <[hidden email]> wrote:
>
>> Based on a cursory reading of FLINK-1297 I would lean toward dropping the
>> code rather than moving to Apache Bahir. This looks to only be appropriate
>> for batch and this module was not integrated into the runtime.
>>
>> If there is a way forward to make use this code in core Flink then that
>> would be even better.
>>
>> On Wed, Oct 19, 2016 at 9:54 AM, Maximilian Michels <[hidden email]>
>> wrote:
>>
>> > +1 for removing it in case it is not widely used. Apache Bahir would
>> > be a more appropriate place for this module then.
>> >
>> > -Max
>> >
>> >
>> > On Wed, Oct 19, 2016 at 3:52 PM, Robert Metzger <[hidden email]>
>> > wrote:
>> > > If there are no users and no contributors of the module, I'm +1 to
>> remove
>> > > it.
>> > >
>> > > If we decide to remove it from flink-contrib, but there are some
>> > > contributors interested in it, I can offer to assist the contributors
>> to
>> > > add the extension to Apache Bahir.
>> > >
>> > > On Wed, Oct 19, 2016 at 2:00 PM, Ufuk Celebi <[hidden email]> wrote:
>> > >
>> > >> Hey devs,
>> > >>
>> > >> I would like to propose the removal of the
>> > >> flink-contrib/flink-operator-stats module.
>> > >>
>> > >> It is currently causing some build stability issues
>> > >> (https://issues.apache.org/jira/browse/FLINK-4833) and there is no
>> > >> active maintainer for it as far as I can tell.
>> > >>
>> > >> Are there any objections to this?
>> > >>
>> > >> I know it always feel bad to remove contributed code, but given the
>> > >> size and scope of the project I think that it is fair to be
>> > >> conservative about these issues. I think we always planned the contrib
>> > >> module to be a staging area where we monitor the usefulness of
>> > >> non-core additions.
>> > >>
>> > >> Best,
>> > >>
>> > >> Ufuk
>> > >>
>> >
>>