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 2021/12/28 15:59:11 UTC

[GitHub] [airflow] potiuk commented on pull request #19736: Add Databricks Deferrable Operators

potiuk commented on pull request #19736:
URL: https://github.com/apache/airflow/pull/19736#issuecomment-1002172319


   This operator will not be clearly importing on Airflow 2.1 (see the errors in tests). Airlfow.trigger is not available in Airlfow 2.1.
   
   There are two approaches we can take:
   
   1) make the Databricks provider  Airlfow 2.2+ 
   2) Try/catch import errors and provide a warning that the Deferrable Operators will only work for Airlfow 2.2+
   
   I am for the 2nd solution as there are quite a few non-defferable operators there that shoudl continue to work. But I would love to hear what others think.
   


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