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

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/02/27 16:52:51 UTC

[GitHub] [airflow] dstandish commented on pull request #20656: Update base sensor operator to support XCOM return value

dstandish commented on pull request #20656:
URL: https://github.com/apache/airflow/pull/20656#issuecomment-1053613393


   > Nice one. Also it has the potential of being detected by our "min airflow vrsion" CI check (It would fail on import).
   > 
   > Anyone else? @dstandish ?
   
   It's good with me.  I recall @potiuk that you were trying to find a way to make it possible to return a tuple.  But I think this change should be compatible with that.  What we _could_ consider doing is, for 3.0, make it so a sensor must be true / false or PRV (i.e., no longer support "truthy" values) and then in 3.1 we could add support for tuple.  WDYT?


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org