itscaro commented on a change in pull request #4056: [AIRFLOW-3207] option to stop task pushing result to xcom URL: https://github.com/apache/incubator-airflow/pull/4056#discussion_r229268278
########## File path: airflow/models.py ########## @@ -2473,6 +2476,7 @@ def __init__( run_as_user=None, task_concurrency=None, executor_config=None, + do_xcom_push=True, Review comment: You can still make it backward compatible by adding `True` to the operators which have this behavior: As what I tried to do in https://github.com/apache/incubator-airflow/pull/3981 Or a branch over this PR: https://github.com/apache/incubator-airflow/compare/master...itscaro:do_xcom_push There is no good to introduce this default behavior as `True` in the models ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services