Skip to content

Conversation

@josh-fell
Copy link
Contributor

@josh-fell josh-fell commented Oct 26, 2021

Closes: #9415, #12726

Adding more color and information about the .output property of operators in the TaskFlow tutorial doc. There are also small changes to keep spelling and descriptions consistent.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

based tasks which need to be run first before a python-based task is run.
Adding dependencies to decorated tasks from traditional tasks
-------------------------------------------------------------
The above tutorial shows how to create dependencies between TaskFlow functions. However, it is
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using the description "TaskFlow function" to be consistent with the description in the Concepts: TaskFlow doc. Let me know if there is a more succinct description we could use.

@github-actions
Copy link

The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide that full test matrix is needed and add the 'full tests needed' label. Then you should rebase it to the latest main or amend the last commit of the PR, and push it with --force-with-lease.

@github-actions github-actions bot added the okay to merge It's ok to merge this PR as it does not require more tests label Oct 26, 2021
Copy link
Contributor

@dstandish dstandish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good but had a few suggestions

@dstandish dstandish merged commit 2dfe85d into apache:main Oct 29, 2021
@josh-fell josh-fell deleted the taskflow-tutorial-output-property branch October 29, 2021 17:22
@jedcunningham jedcunningham added the type:doc-only Changelog: Doc Only label Dec 7, 2021
@jedcunningham jedcunningham added this to the Airflow 2.2.3 milestone Dec 7, 2021
jedcunningham pushed a commit that referenced this pull request Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind:documentation okay to merge It's ok to merge this PR as it does not require more tests type:doc-only Changelog: Doc Only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[AIP-31] Update DAG examples to use Functional DAG style

4 participants