pyairtable icon indicating copy to clipboard operation
pyairtable copied to clipboard

Parameterize max records per request

Open lucas-simonelli opened this issue 3 years ago • 4 comments
trafficstars

I'm pushing a lot of records to a table (~10k), so I'd rather try more than 10 per request in the batch creation/updates. This PR parameterized the MAX_RECORD_PER_REQUEST arg, defaulting to 10

lucas-simonelli avatar Mar 02 '22 17:03 lucas-simonelli

@gtalarico not sure what the contributing guidelines are, LMK if i missed something 👍

lucas-simonelli avatar Mar 03 '22 10:03 lucas-simonelli

Hey @gtalarico – I will actually need this PR also in a side project of mine! :) Would mind taking a look when you have a moment?

kkom avatar Mar 20 '22 14:03 kkom

@kkom There are still some unresolved issues above. And I will still need to tag and ship after, so unfortunately this might not be ready for when you needed!

However, you can install and add dep from a branch/commit, so hopefully you can use that as a temporary fix 👇 https://stackoverflow.com/questions/16584552/how-to-state-in-requirements-txt-a-direct-github-source https://stackoverflow.com/questions/20101834/pip-install-from-git-repo-branch

gtalarico avatar Mar 21 '22 23:03 gtalarico

No worries, I can wait a little. It turned out that my side project is fast enough for now anyway, so there is no terrible rush. Thanks @gtalarico !

kkom avatar Mar 21 '22 23:03 kkom

Hi there, I'm going through old PRs, and I'm wondering if you've found a way to make this sort of thing work?.When I try to pass more than 10 records at a time to one of the batch APIs, I get a 422 error:

{
  "type": "INVALID_RECORDS",
  "message": "A maximum of 10 records can be created per request but you have provided 100."
}

Since more than 10 records is explicitly not supported and does not appear to work, can I close this PR?

mesozoic avatar Apr 22 '23 06:04 mesozoic

Might be that they changed the API limit after all this time. Closing it then!

lucas-simonelli avatar Apr 24 '23 08:04 lucas-simonelli