Dev-iL commented on code in PR #37028:
URL: https://github.com/apache/airflow/pull/37028#discussion_r1632299745
##########
docs/apache-airflow/core-concepts/operators.rst:
##########
@@ -211,64 +235,65 @@ Alternatively, if you want to prevent Airflow from
treating a value as a referen
Rendering Fields as Native Python Objects
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
-By default, all the ``template_fields`` are rendered as strings.
-
-Example, let's say ``extract`` task pushes a dictionary
-(Example: ``{"1001": 301.27, "1002": 433.21, "1003": 502.22}``) to :ref:`XCom
<concepts:xcom>` table.
-Now, when the following task is run, ``order_data`` argument is passed a
string, example:
-``'{"1001": 301.27, "1002": 433.21, "1003": 502.22}'``.
+By default, all Jinja templates in ``template_fields`` are rendered as
strings. This however not always desired. For example, let's say an ``extract``
task pushes a dictionary ``{"1001": 301.27, "1002": 433.21, "1003": 502.22}``
to :ref:`XCom <concepts:xcom>`:
Review Comment:
The sentence "_This however not always desired_." is grammatically
incorrect, it needs an "is".
(It is probably better to add commas around _however_ in that sentence, but
that's not as important IMHO.)
--
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]