-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
🐛 Source Google Analytics 4 (GA4): improve rate limits messages and retry logic #23822
🐛 Source Google Analytics 4 (GA4): improve rate limits messages and retry logic #23822
Conversation
…nalytics-4-fix-rate-limits
/test connector=connectors/source-google-analytics-data-api
Build PassedTest summary info:
|
airbyte-integrations/connectors/source-google-analytics-data-api/setup.py
Show resolved
Hide resolved
airbyte-integrations/connectors/source-google-analytics-data-api/acceptance-test-config.yml
Show resolved
Hide resolved
/publish connector=connectors/source-google-analytics-data-api
if you have connectors that successfully published but failed definition generation, follow step 4 here |
@evantahler please advise on this failing test: When you open the actual failed test, you will see it's succeeded: Can I merge in safely with this one failing? |
You can ignore the |
What
Resolving:
How
GoogleAnalyticsApiQuota
to handle quota-specific scenarios, based onerror message
(not all cases covered. The other cases could be added easily having the error message later on)Recommended reading order
Starting with
source.py > should_retry
- values for
should_retry
,raise_on_http_errors
are referenced from theGoogleAnalyticsApiQuota
to cover the scenarios forshould_retry
- value for
backoff_time
is referenced from theGoogleAnalyticsApiQuota
to cover the scenarios forbackoff_time
Handled scenarios:
🚨 User Impact 🚨
No impact is expected.
Pre-merge Checklist
Expand the relevant checklist and delete the others.
Updating a connector
Community member or Airbyter
airbyte_secret
./gradlew :airbyte-integrations:connectors:<name>:integrationTest
.README.md
bootstrap.md
. See description and examplesdocs/integrations/<source or destination>/<name>.md
including changelog. See changelog exampleAirbyter
If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.
/test connector=connectors/<name>
command is passing/publish
command described here