You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "Ash Berlin-Taylor (JIRA)" <ji...@apache.org> on 2018/10/01 10:08:00 UTC

[jira] [Updated] (AIRFLOW-3133) Don't `return True` for operators

     [ https://issues.apache.org/jira/browse/AIRFLOW-3133?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ash Berlin-Taylor updated AIRFLOW-3133:
---------------------------------------
    Description: 
disable `return True` for SSHOperator since there is already a flag to push to XCOM the stdout, pushing True to XCOM automatically might be too noisy for XCOM database.

There are other operators that do this as well, and it doesn't provide any value - it just creates an entry in the XCom table that is redundant - we already know that the task succeeded.

  was:disable `return True` for SSHOperator since there is already a flag to push to XCOM the stdout, pushing True to XCOM automatically might be too noisy for XCOM database

        Summary: Don't `return True` for operators  (was: [Proposal] Disable `return True` for SSHOperator)

> Don't `return True` for operators
> ---------------------------------
>
>                 Key: AIRFLOW-3133
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-3133
>             Project: Apache Airflow
>          Issue Type: Improvement
>          Components: contrib
>    Affects Versions: 1.10.0
>            Reporter: Minh Quan TRAN
>            Priority: Minor
>
> disable `return True` for SSHOperator since there is already a flag to push to XCOM the stdout, pushing True to XCOM automatically might be too noisy for XCOM database.
> There are other operators that do this as well, and it doesn't provide any value - it just creates an entry in the XCom table that is redundant - we already know that the task succeeded.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)