potiuk commented on issue #28405:
URL: https://github.com/apache/airflow/issues/28405#issuecomment-1359649621

   @dinigo I believe this is exactly what has been discussed in #12611 . 
   
   Those two concepts (of "per-task notification" and "airflow system-wide 
notification") are easilly confused and it happened already in #12611. So let's 
not mix those two - if you think you want "general notification" you should 
discuss it there. 
   
   BUT
   
   The #12611 has been closed (by me) because I believe Airflow 2.5.0 
"listener" API 
https://airflow.apache.org/docs/apache-airflow/stable/listeners.html  allows 
you to do exactly what you want. You can write your own notifiers, as comples 
or as difficult you want and using whatever mechanism you wantt for 
notification - by plugging in your own listener. You can even contribute such 
listeners back to Airflow if you think they might be useful for others (and for 
example add them to providers - similarly like elasticsearch, cloudwatch etc. 
logging handlers have been added there).  While listener API has not been 
foreseen for this initially, I believe it perfectly fits the needs you have. I 
suggest you migrate to Airflow 2.5.0 and try it even now to see if it works.  
If you will find otherwise - feel free to comment in #12611 about your findings 
and why you think it's not good enough.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to