@itscaro Yup, this looks good. Other operators should probably not `return 
true` either. Do you want to update this PR to remove the other uses of `return 
true`?

As for the ones that return - we should probably put those behind an 
`xcom_push` flag with a default value of True (so that the usage of the 
operator doesn't change). Again, would you like to do that here? Or we can 
create an issue and someone else could pick that up if you don't have 
time/desire.

[ Full content available at: 
https://github.com/apache/incubator-airflow/pull/3981 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to