uranusjr commented on code in PR #41744:
URL: https://github.com/apache/airflow/pull/41744#discussion_r1730956484
##########
airflow/models/baseoperator.py:
##########
@@ -714,12 +714,14 @@ class derived from this one results in the creation of a
task object,
execution, receiving a context dictionary; raising an exception will
prevent the task from being executed.
- |experimental|
+ |experimental| Note that passing a function to the pre_execute
parameter is experimental,
+ while overriding .pre_execute in a custom operator is stable.
Review Comment:
```suggestion
|experimental| Passing a function to this argument is experimental,
although
(re-)implementing ``pre_execute`` in a custom operator subclass is
stable.
```
This is probably a little better? It might still be confusing to some users
what *overriding* means. Not sure TBH.
--
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]
