+1, but I think we still need the migration script for the change. On Wed, Jan 2, 2019 at 1:44 PM Bas Harenslak <basharens...@godatadriven.com> wrote:
> +1 > > > On 2 Jan 2019, at 22:42, Daniel Imberman <daniel.imber...@gmail.com> > wrote: > > > > +1 > > > > On Wed, Jan 2, 2019 at 1:18 PM Maxime Beauchemin < > maximebeauche...@gmail.com> > > wrote: > > > >> +1 > >> > >> On Wed, Jan 2, 2019 at 1:00 PM Dan Davydov <ddavy...@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 > >>>> > >>> > >> > >