singer-io / tap-marketo

GNU Affero General Public License v3.0

9 stars 17 forks source link

Add retry to handle 20 sec quota limit #60

Closed dmosorast closed 4 years ago

dmosorast commented 4 years ago

Description of change

Marketo has a short term quota limit of 100 requests per 20 seconds across all apps using a single Marketo account (or so it seems).

This results in a 606 error code, so this PR will handle that and backoff so as to not interrupt the sync job.

Manual QA steps

Implemented from a specific example and fits in with the current pattern, could not reproduce at will, so testing in a patch version.

Risks

Low, it's additive to the error handling. Worst case it doesn't catch the error.

Rollback steps

revert this branch and bump version