eladkal commented on PR #34257: URL: https://github.com/apache/airflow/pull/34257#issuecomment-1712928516
I am not convinced yet. We should increase min versions only when absolutely necessary and we can demonstrate clear value by doing so. The azure package had a clear value - introducing of new functionality and unblocking users who want to use it. In this case we have nothing to say about why we are setting higher lower limit. You mentioned the conflict part - but that is not the only thing. Providers may be updated seperatly from core. If user will bump google provider you are now also force him to update common.sql provider. That requires more checks, more testing, more time and what if this version of sql provider contains a bug? It happens... Even if we say "everything should be the same" some users take the caution way and sometimes its just the procedure. We should recognize this. I myself been there. I prefer to let users decide when they want to update each provider and not force them into doing something they did not intended unless there is a very good reason. That said I am -0 here so feel free to merge this one. Mainly because it's common.sql I wrote this comment just to clarify that what we are going to do here should not be used as the "go to" fix for other providers. -- 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]
