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/28 12:11:24 UTC

[GitHub] [airflow] uranusjr commented on pull request #21815: Make BaseSensorOperator.deps a class variable

uranusjr commented on pull request #21815:
URL: https://github.com/apache/airflow/pull/21815#issuecomment-1054193428


   > may break/not work with customer operators if someone sets deps on an instance level :(
   
   Maybe we can detect that (easy enough, simply check the attribute is actually a set/collection on the class level and not a descriptor), and raise an error saying the operator needs to be fixed? This can likely be done on DAG-parsing time.


-- 
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