infused-kim commented on issue #17111: URL: https://github.com/apache/airflow/issues/17111#issuecomment-884863829
The PR above makes the hook compatible with newer versions of the google-ads API. The biggest breaking change in the google-ads API was that instead of returning results as native protocol buffers, it's now returning the data in a simpler to use proto-plus format. To ensure backwards compatibility for older operators that use the hook, I made the search() method convert the data back into native protocol buffers. In addition to that I added a search_proto_plus() method that returns the data in the new proto-plus format. You can get more information about that here: https://developers.google.com/google-ads/api/docs/client-libs/python/library-version-10 ## Prioritization thoughts Google completely stopped supporting the v5 API as of today. This means that the integrations of everyone who is using this hook are broken now. So perhaps it's worth considering to do an expedited release of the update to the google provider. -- 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]
