tap-adwords rate limit I was able to run on `ads`...
# plugins-general
s
tap-adwords rate limit I was able to run on
ads
and
AD_PERFORMANCE_REPORTS
successfully when the start date is set to
27-Mar
. But anything realistic like 3 months before causes a rate limit failure (daily limit exceeded for Basic account) on just these schema objects. Is there a batch parameter I can tune to reduce the number of overall requests.
t
that’d have to be something the tap supports
s
I can't find it documented in Meltano's docs or in the tap repo. Any way to get in touch with the original authors?
t
@subhash_gopalakrishnan apologies for the delayed response. The Meltano variant of the tap is forked from this Singer repo https://github.com/singer-io/tap-adwords Looking at the issue tracker it seems they’re fairly unresponsive. We’re more than happy to review a contribution though to https://gitlab.com/meltano/tap-adwords to add this functionality
s
Thanks for the response, @taylor I will certainly consider contributing a fix. To better understand the issue, here are some quick questions. It'll be great to hear your thoughts on the same: 1. The generic entities are being queried at a constant page size. Any specific motivation for this? Any harm in making this configurable? 2. Reports are always queried for one day at a time. Will it make sense to provide the start and end dates as part of a single query instead of making multiple queries for each day? Again, this seems to be intentional - so maybe I am missing something?
t
Making the page size configurable makes sense to me. As long as the current behavior is the default that should be no problem to add.
To your second question, I don’t have enough experience with this source to know if the single day query is intentional - but it seems logical to me to make fewer api calls by running it as a single query. I glance through the api docs to see if anything jumps out at me though
nothing immediately jumps out at me, but the docs do talk about the page size being configurable, so that’s a good first iteration at least
s
👍