You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
CSV/Airtable data imports trigger lots of audit changes which causes timeouts due to concurrent requests. (fails at when 100+ concurrent requests with minimal setup)
Possible solution:
I'd suggest extending this endpoint to process an array of audit changes to fix failing requests.
const{ audits }=req.body;if(!Array.isArray(audits)){thrownewError("400: Bad Request");}returnPromise.all(audits.map(audit=>log.write(log.entry(metadata,audit))))
The text was updated successfully, but these errors were encountered:
htuerker
changed the title
Log many audit changes within single request
Log many audit changes within a request
Aug 18, 2022
Problem:
CSV/Airtable data imports trigger lots of audit changes which causes timeouts due to concurrent requests. (fails at when 100+ concurrent requests with minimal setup)
Possible solution:
I'd suggest extending this endpoint to process an array of audit changes to fix failing requests.
The front-end consumer is here:
https://github.com/rowyio/rowy/blob/a35afa96199473ce9e73e30f41a815ed35f895cc/src/sources/TableSourceFirestore/useAuditChange.ts#L43-L64
We can make requests with "audits" as an array here. The only drawback here is we have to change the current usage(only 4-5 places)
In client:
Here we can write logs iteratively.
The text was updated successfully, but these errors were encountered: