kaxil commented on a change in pull request #4071: [AIRFLOW-3237] Refactor example DAGs URL: https://github.com/apache/incubator-airflow/pull/4071#discussion_r227667674
########## File path: airflow/example_dags/example_xcom.py ########## @@ -63,12 +61,21 @@ def puller(**kwargs): push1 = PythonOperator( - task_id='push', dag=dag, python_callable=push) + task_id='push', + dag=dag, + python_callable=push, +) push2 = PythonOperator( - task_id='push_by_returning', dag=dag, python_callable=push_by_returning) + task_id='push_by_returning', + dag=dag, + python_callable=push_by_returning, +) pull = PythonOperator( - task_id='puller', dag=dag, python_callable=puller) + task_id='puller', + dag=dag, + python_callable=puller, +) -pull.set_upstream([push1, push2]) +pull << [push1, push2] Review comment: @XD-DENG Well, I was unsure of the task list usage with rshift and lshift operators😂 not `>>` or `<<` (rshift or lshift) itself... I think we already had good documentation over using them and I prefer using those as it resembles DAG + it is more clean. Many of our examples already do that. But yes after @BasPH pointed o the implementation we can surely do that and should be documented. @BasPH can you please add the docs. One liner with one or two examples is fine. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
