You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@airflow.apache.org by "Driesprong, Fokko" <fo...@driesprong.frl> on 2019/01/02 20:37:51 UTC
[DISCUSS] Deprecate KnownEvent/KnownEventType
Hi all,
Recently I've opened up a PR
<https://github.com/apache/incubator-airflow/pull/4421> to remove the
KnownEvent and KnownEventType for Apache Airflow 2.0. My feeling was that
not a lot of people are using this functionality, and therefore I would
like to remove it to simplify the codebase.
Are there any people on the mailing list which are actually using this
functionality. Please let me know.
Cheers, Fokko
Re: [DISCUSS] Deprecate KnownEvent/KnownEventType
Posted by Felix Uellendall <fe...@gmx.de>.
+1
Am 02/01/2019 um 22:52 schrieb Tao Feng:
> +1, but I think we still need the migration script for the change.
>
> On Wed, Jan 2, 2019 at 1:44 PM Bas Harenslak <ba...@godatadriven.com>
> wrote:
>
>> +1
>>
>>> On 2 Jan 2019, at 22:42, Daniel Imberman <da...@gmail.com>
>> wrote:
>>> +1
>>>
>>> On Wed, Jan 2, 2019 at 1:18 PM Maxime Beauchemin <
>> maximebeauchemin@gmail.com>
>>> wrote:
>>>
>>>> +1
>>>>
>>>> On Wed, Jan 2, 2019 at 1:00 PM Dan Davydov <ddavydov@twitter.com.invalid
>>>> wrote:
>>>>
>>>>> +1 to removing
>>>>>
>>>>> On Wed, Jan 2, 2019 at 10:48 PM Driesprong, Fokko <fokko@driesprong.frl
>>>>> wrote:
>>>>>
>>>>>> Hi all,
>>>>>>
>>>>>> Recently I've opened up a PR
>>>>>> <https://github.com/apache/incubator-airflow/pull/4421> to remove the
>>>>>> KnownEvent and KnownEventType for Apache Airflow 2.0. My feeling was
>>>> that
>>>>>> not a lot of people are using this functionality, and therefore I
>> would
>>>>>> like to remove it to simplify the codebase.
>>>>>> Are there any people on the mailing list which are actually using this
>>>>>> functionality. Please let me know.
>>>>>>
>>>>>> Cheers, Fokko
>>>>>>
>>
Re: [DISCUSS] Deprecate KnownEvent/KnownEventType
Posted by Tao Feng <fe...@gmail.com>.
+1, but I think we still need the migration script for the change.
On Wed, Jan 2, 2019 at 1:44 PM Bas Harenslak <ba...@godatadriven.com>
wrote:
> +1
>
> > On 2 Jan 2019, at 22:42, Daniel Imberman <da...@gmail.com>
> wrote:
> >
> > +1
> >
> > On Wed, Jan 2, 2019 at 1:18 PM Maxime Beauchemin <
> maximebeauchemin@gmail.com>
> > wrote:
> >
> >> +1
> >>
> >> On Wed, Jan 2, 2019 at 1:00 PM Dan Davydov <ddavydov@twitter.com.invalid
> >
> >> wrote:
> >>
> >>> +1 to removing
> >>>
> >>> On Wed, Jan 2, 2019 at 10:48 PM Driesprong, Fokko <fokko@driesprong.frl
> >
> >>> wrote:
> >>>
> >>>> Hi all,
> >>>>
> >>>> Recently I've opened up a PR
> >>>> <https://github.com/apache/incubator-airflow/pull/4421> to remove the
> >>>> KnownEvent and KnownEventType for Apache Airflow 2.0. My feeling was
> >> that
> >>>> not a lot of people are using this functionality, and therefore I
> would
> >>>> like to remove it to simplify the codebase.
> >>>> Are there any people on the mailing list which are actually using this
> >>>> functionality. Please let me know.
> >>>>
> >>>> Cheers, Fokko
> >>>>
> >>>
> >>
>
>
Re: [DISCUSS] Deprecate KnownEvent/KnownEventType
Posted by Bas Harenslak <ba...@godatadriven.com>.
+1
> On 2 Jan 2019, at 22:42, Daniel Imberman <da...@gmail.com> wrote:
>
> +1
>
> On Wed, Jan 2, 2019 at 1:18 PM Maxime Beauchemin <ma...@gmail.com>
> wrote:
>
>> +1
>>
>> On Wed, Jan 2, 2019 at 1:00 PM Dan Davydov <dd...@twitter.com.invalid>
>> wrote:
>>
>>> +1 to removing
>>>
>>> On Wed, Jan 2, 2019 at 10:48 PM Driesprong, Fokko <fo...@driesprong.frl>
>>> wrote:
>>>
>>>> Hi all,
>>>>
>>>> Recently I've opened up a PR
>>>> <https://github.com/apache/incubator-airflow/pull/4421> to remove the
>>>> KnownEvent and KnownEventType for Apache Airflow 2.0. My feeling was
>> that
>>>> not a lot of people are using this functionality, and therefore I would
>>>> like to remove it to simplify the codebase.
>>>> Are there any people on the mailing list which are actually using this
>>>> functionality. Please let me know.
>>>>
>>>> Cheers, Fokko
>>>>
>>>
>>
Re: [DISCUSS] Deprecate KnownEvent/KnownEventType
Posted by Daniel Imberman <da...@gmail.com>.
+1
On Wed, Jan 2, 2019 at 1:18 PM Maxime Beauchemin <ma...@gmail.com>
wrote:
> +1
>
> On Wed, Jan 2, 2019 at 1:00 PM Dan Davydov <dd...@twitter.com.invalid>
> wrote:
>
> > +1 to removing
> >
> > On Wed, Jan 2, 2019 at 10:48 PM Driesprong, Fokko <fo...@driesprong.frl>
> > wrote:
> >
> > > Hi all,
> > >
> > > Recently I've opened up a PR
> > > <https://github.com/apache/incubator-airflow/pull/4421> to remove the
> > > KnownEvent and KnownEventType for Apache Airflow 2.0. My feeling was
> that
> > > not a lot of people are using this functionality, and therefore I would
> > > like to remove it to simplify the codebase.
> > > Are there any people on the mailing list which are actually using this
> > > functionality. Please let me know.
> > >
> > > Cheers, Fokko
> > >
> >
>
Re: [DISCUSS] Deprecate KnownEvent/KnownEventType
Posted by Maxime Beauchemin <ma...@gmail.com>.
+1
On Wed, Jan 2, 2019 at 1:00 PM Dan Davydov <dd...@twitter.com.invalid>
wrote:
> +1 to removing
>
> On Wed, Jan 2, 2019 at 10:48 PM Driesprong, Fokko <fo...@driesprong.frl>
> wrote:
>
> > Hi all,
> >
> > Recently I've opened up a PR
> > <https://github.com/apache/incubator-airflow/pull/4421> to remove the
> > KnownEvent and KnownEventType for Apache Airflow 2.0. My feeling was that
> > not a lot of people are using this functionality, and therefore I would
> > like to remove it to simplify the codebase.
> > Are there any people on the mailing list which are actually using this
> > functionality. Please let me know.
> >
> > Cheers, Fokko
> >
>
Re: [DISCUSS] Deprecate KnownEvent/KnownEventType
Posted by Dan Davydov <dd...@twitter.com.INVALID>.
+1 to removing
On Wed, Jan 2, 2019 at 10:48 PM Driesprong, Fokko <fo...@driesprong.frl>
wrote:
> Hi all,
>
> Recently I've opened up a PR
> <https://github.com/apache/incubator-airflow/pull/4421> to remove the
> KnownEvent and KnownEventType for Apache Airflow 2.0. My feeling was that
> not a lot of people are using this functionality, and therefore I would
> like to remove it to simplify the codebase.
> Are there any people on the mailing list which are actually using this
> functionality. Please let me know.
>
> Cheers, Fokko
>