potiuk commented on issue #25721: URL: https://github.com/apache/airflow/issues/25721#issuecomment-1216610742
Would be great @alexott if you could test the Databricks provider together with the new common.sql :). Also I found one potential issue with default 'split_sql` set to True - the 3.2.0rc4 new databricks provider actually depends on the 1.1.0 feature of "common-sql' - but we do not have min-requirement for it (for other providers it's non-default feature so they do not REALLY depend on it). It's very unlikely for someone to get them not upgraded at the same time, but this is a posibility. I do not think this is worth rc5 - rather than that I'd put common-sql>= 1.1.0 in the next version. But maybe it's worth delaying the release of databricks and do it `well`. WDYT @alexott @kazanzhy ? -- 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]
