dabla commented on PR #40665:
URL: https://github.com/apache/airflow/pull/40665#issuecomment-2225592619

   > > The only drawback of this change is that all providers then need to 
update the min required version of the common.sql provider to be at the latest 
version.
   > 
   > This is absolutely fine (precisely because we intend to keep strict 
backwards compatibility for all common.sql versions). It's a bit 
**unfortunate** that 1.14.1 was not 1.15.0 for that one (because it added 
`get_conn_id()` so it could be classified as new feature) - but this is just a 
small inconvenience and that ship has sailed :)
   
   Sorry for that, so I should have done a minor bump then for the common sql 
provider before, good to know as I will need to do that for the next PR also.  
This time will try to do it correctly


-- 
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: [email protected]

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

Reply via email to