[GitHub] [airflow] potiuk commented on a change in pull request #20640: Update operators.rst

[GitBox <gi...@apache.org>]

potiuk commented on a change in pull request #20640:
URL: https://github.com/apache/airflow/pull/20640#discussion_r777904597



##########
File path: docs/apache-airflow/concepts/operators.rst
##########
@@ -208,3 +208,21 @@ In this case, ``order_data`` argument is passed: ``{"1001": 301.27, "1002": 433.
 Airflow uses Jinja's `NativeEnvironment <https://jinja.palletsprojects.com/en/2.11.x/nativetypes/>`_
 when ``render_template_as_native_obj`` is set to ``True``.
 With ``NativeEnvironment``, rendering a template produces a native Python type.
+
+.. _concepts:reserved-keywords:
+
+Reserved params keyword
+-----------------------
+
+In Apache Airflow 2.3.3 ``params`` variable is used during DAG serialization. Please do not use that name in third party operators.
+If you upgrade your environment and get the following error
+
+.. code-block:: python

Review comment:
       I tihnk `params` is quite likely name that will be used by custom operators, so having a warning against this one (and the fact that it has been **just** added make it more likely for people to get the same problem.
   
   Simply - operators that worked in 2.1 will stop working in 2.2 - which is kinda unexpected. So I think this message is useful




-- 
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-unsubscribe@airflow.apache.org

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