potiuk commented on issue #27838: URL: https://github.com/apache/airflow/issues/27838#issuecomment-1323945292
The issue is with how past bigquery providers mis-used protected method of the common-sql provider, indeed. Unfortunately, the only way to fix it, is to bring the method back in 1.3.1 and yank common.sql 1.3.0, because this mis-use made common.sql provider 1.3.0 backwards-incompatible. I am afraid it is on us rather than on Composer team, because the problem is with the google provider versions we release and maintain. We cannot break older released google provider versions by installing new version of common.sql provider which is not backwrds-compatible (and in this case unfortunately OUR google provider made use of something that OUR common.sql provider thought was an internal detail. In any case it is OUR problem to solve. I will release common.sql It is what it is, unfortunately, and one more learning that in case of common code like that we need to be extra careful and in the future we should use `__` (double underscore) methods for internal commnds rather than `_` because the `_` methods are not sufficiently protected agains accidental mis-use. -- 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]
