jscheffl commented on code in PR #36374:
URL: https://github.com/apache/airflow/pull/36374#discussion_r1438943838


##########
docs/apache-airflow/administration-and-deployment/logging-monitoring/callbacks.rst:
##########
@@ -49,6 +49,7 @@ Name                                        Description
 ``sla_miss_callback``                       Invoked when a task misses its 
defined :ref:`SLA <concepts:slas>`
 ``on_retry_callback``                       Invoked when the task is :ref:`up 
for retry <concepts:task-instances>`
 ``on_execute_callback``                     Invoked right before the task 
begins executing.
+``on_skipped_callback``                     Invoked when the task is 
:ref:`running <concepts:task-instances>` and  AirflowSkipException raised.

Review Comment:
   +1 here:
   ```suggestion
   ``on_skipped_callback``                     Invoked when the task is 
:ref:`running <concepts:task-instances>` and  AirflowSkipException raised.
                                               Explicitly it is NOT called if a 
task is not started to be executed because of a preceding branching
                                               decision in the DAG or a trigger 
rule which causes execution to skip so that the task execution
                                               is never scheduled.
   ```



-- 
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: commits-unsubscr...@airflow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to