Hi Daniel,

I've actually implemented an AirflowNotifier base class for others to
extend. This would allow for custom notifiers.

Niels

Op 5 jul. 2017 8:57 p.m. schreef "Daniel Imberman" <
[email protected]>:

Hi Niels,

Thank you for setting this up! I think this is a feature that would really
useful to a lot of people. Have you considered setting up a "notifier
plugin" so that people can create custom notifiers? The API seems pretty
consistent so I don't think it would be too much work to add.

On Wed, Jul 5, 2017 at 11:40 AM Niels Zeilemaker <
[email protected]> wrote:

> Hi All,
>
> I've opened a pull request (
> https://github.com/apache/incubator-airflow/pull/2412) which introduces
> the concept of notifiers.
> I've made this change as I have a requirement to push status changes of
> failed/retried jobs to more than only email. Eg, I want to use slack in
> this case.
> Currently, we can only pass an email notifier, which I feel is pretty
> limited.
>
> My pull-request changes this and introduces Notifiers, which are triggered
> on status changes. I think it's almost there, but I have some problems
> trying to migrate the SLA stuff into the same concept. Let me know if you
> have any comments. Or if you feel this is the wrong approach to my
problem.
>
> Niels
>

Reply via email to