You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/04/04 15:23:01 UTC

[GitHub] [airflow] marcosmarxm commented on a change in pull request #15183: Move common pitfall documentation to Airflow docs

marcosmarxm commented on a change in pull request #15183:
URL: https://github.com/apache/airflow/pull/15183#discussion_r606816164



##########
File path: docs/apache-airflow/best-practices.rst
##########
@@ -54,6 +59,12 @@ Some of the ways you can avoid producing a different result -
     You should define repetitive parameters such as ``connection_id`` or S3 paths in ``default_args`` rather than declaring them for each task.
     The ``default_args`` help to avoid mistakes such as typographical errors.
 
+Creating a custom Operator
+---------------------------
+
+When implementing custom operators, do not make any expensive expensive operations in their ``__init__``. They are

Review comment:
       ```suggestion
   When implementing custom operators, do not make any expensive operations in their ``__init__``.
   ```
   Small typo




-- 
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.

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