planet-client-python icon indicating copy to clipboard operation
planet-client-python copied to clipboard

update session limiting to take advantage of new 'burst' support in APIs

Open jreiberkyle opened this issue 3 years ago • 1 comments

The APIs have been updated to support 'bursts' of requests. This means that TooManyRequests should no longer be returned if the number of initial requests is below the rate limit. Therefore, the current custom implementation of _Limit, which enforces a delay between requests, is likely no longer needed.

Try replacing _Limiter with asyncio-throttle and rerun session_configuration.py. Make note of total run time vs original and number of TooManyRequests errors. Given improved performance, implement the change and update docs.

jreiberkyle avatar Sep 15 '22 19:09 jreiberkyle

This may help with the problems described in #1050 as well

tbarsballe avatar Aug 08 '24 19:08 tbarsballe