Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Workaround backwards-incompatibility in google-core-api 2.16.0 #37093

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Jan 30, 2024

There is a backwards-incompatible change in
google.api_core.retry.AsyncRetry imports.

In 2.16.0 version of google-api-core, AsyncRetry was moved to google.api_core.retry_unary_async and backwards compatibility impots were not haandling the case of:
from google.api_core.retry_async imprt AsyncRetry

The issue is tracked in googleapis/python-api-core#586

Until it is solved, we need to handle both cases, because one works before and one after 2.16.0.

But there is no import that works for both.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

There is a backwards-incompatible change in
google.api_core.retry.AsyncRetry imports.

In 2.16.0 version of google-api-core, AsyncRetry was moved to
google.api_core.retry_unary_async and backwards compatibility impots
were not haandling the case of:
`from google.api_core.retry_async imprt AsyncRetry`

The issue is tracked in googleapis/python-api-core#586

Until it is solved, we need to handle both cases, because one works
before and one after 2.16.0.

But there is no import that works for both.
@boring-cyborg boring-cyborg bot added area:providers area:system-tests provider:google Google (including GCP) related issues labels Jan 30, 2024
Copy link
Contributor

@amoghrajesh amoghrajesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@potiuk potiuk merged commit 762f544 into apache:main Jan 30, 2024
13 of 14 checks passed
@potiuk potiuk deleted the workaround-google-core-api-backwards-incompatibility-issue branch January 30, 2024 12:32
abhishekbhakat pushed a commit to abhishekbhakat/my_airflow that referenced this pull request Mar 5, 2024
…e#37093)

There is a backwards-incompatible change in
google.api_core.retry.AsyncRetry imports.

In 2.16.0 version of google-api-core, AsyncRetry was moved to
google.api_core.retry_unary_async and backwards compatibility impots
were not haandling the case of:
`from google.api_core.retry_async imprt AsyncRetry`

The issue is tracked in googleapis/python-api-core#586

Until it is solved, we need to handle both cases, because one works
before and one after 2.16.0.

But there is no import that works for both.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers area:system-tests provider:google Google (including GCP) related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants