potiuk commented on issue #25721: URL: https://github.com/apache/airflow/issues/25721#issuecomment-1216631065
Actually seems that more providers are afffected - see https://github.com/apache/airflow/issues/25740 . I have not realised we have another implicit dependency there as >= 1.1.0 should also be added for all other providers also using `strip_sql_string`. This will get fixed for those who upgrade to 1.1.0 of comon sql (once we release) but I think we should bump all the providers that depend on common-sql to have >=1.1.0 - also just in case we missed some other implicit dependency. BTW. Those are the kind of problems i anticipated with common-sql. Managing those kind of cross-dependencies is something that I think we should be more careful about in general. But this is a good learning for the future when we do more of this kind of approach and maybe I will figure some kind of automation to help with that. I will think of that :) @alexott -> I think this is quite ok to release this RC4 now without the limit, but right after this vote is complete, I think I will release a new wave of sql-only providers that do not have the common-sql limit to bump it up (including the databricks 3.2.1) WDYT ? -- 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]
