I left a few comments on the PR. Note that this approach is marginally more flexible/expressive to what can already be accomplished with `on_success_callback`, `on_failure_callback` and `on_retry_callback`.
On Wed, Jul 5, 2017 at 7:03 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 >
