BasPH commented on PR #26486: URL: https://github.com/apache/airflow/pull/26486#issuecomment-1251541941
I'm hesitant to show the TaskFlow style in this example because decorators are typically not learned on day 1 of Python. The home page should be understandable to all audiences. Support for TaskFlow style operators is still limited. You can't run Bash code using the TaskFlow API, causing this example to be unnecessarily complex when converted to the TaskFlow API because you have to mix concepts: "standard" style operators and "taskflow" style operators to achieve the same. Also, I think the beauty of the example in this PR is that it demonstrates the ability to run tasks using different technologies which is one of Airflow's key strengths. If you were to convert this example to a full TaskFlow API style example, you'd have two PythonOperators. Additionally, this proposed (or put back) code sample demonstrates nicely in the UI: you clearly see one BashOperator, and one PythonOperator. Using the TaskFlow operators displays `_PythonDecoratedOperator` in the UI which is not self-explanatory IMO. Not trying to bring down the TaskFlow API but I think it's better introduced elsewhere. -- 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]
