eladkal commented on pull request #15533:
URL: https://github.com/apache/airflow/pull/15533#issuecomment-830185578


   > LGTM. @eladkal - are you ok with it in the context of other dbs? I think 
any DBHook change will have to be done as part of 2.1 but maybe this is a good 
start to implement it "per operator" and then generalise the approach for 2.1 
(and then we could release versions of providers that will be 2.1+ compatible 
only when we have the generalisation in 2.1- as we discussed that we will do at 
some point in time.
   
   It's OK to do it per operator as well
   


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to