josh-fell commented on a change in pull request #19214:
URL: https://github.com/apache/airflow/pull/19214#discussion_r736501487
##########
File path: docs/apache-airflow/tutorial_taskflow_api.rst
##########
@@ -251,18 +251,41 @@ Building this dependency is shown in the code below:
file_task >> order_data
-In the above code block, a new python-based task is defined as
``extract_from_file`` which
+In the above code block, a new TaskFlow function is defined as
``extract_from_file`` which
reads the data from a known file location.
In the main DAG, a new ``FileSensor`` task is defined to check for this file.
Please note
that this is a Sensor task which waits for the file.
-Finally, a dependency between this Sensor task and the python-based task is
specified.
+Finally, a dependency between this Sensor task and the TaskFlow function is
specified.
-Consuming XCOMs with decorated tasks from regular tasks
----------------------------------------------------------
-You may additionally find it necessary to consume an XCOM from a pre-existing
task as an input into python-based tasks.
+Consuming XComs from traditional tasks as inputs
+------------------------------------------------
+You may find it necessary to consume an XCom from traditional task, either
pushed within the task's execution
+or via its return value, as an input into downstream tasks. You can access the
pushed XCom (also known as an
Review comment:
Agreed it's more direct but interacting with users in Slack and
StackOverflow it doesn't seem particularly clear, especially to new users, that
`XComs` can be pushed via those 2 methods: manually pushed in a task + a return
value. Being a little bit verbose here might be beneficial in this case but not
a hill I'm going to die on 😄
--
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]