Skip to content
This repository has been archived by the owner on Nov 10, 2021. It is now read-only.

Throttling logic #15

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

anonymoussprocket
Copy link
Contributor

Adjust pThrottle({ limit: 15, interval: 1200 }) to avoid the issue, note that it may increase load times.

@veqtor
Copy link
Collaborator

veqtor commented Apr 6, 2021

How may it increase load times?

@anonymoussprocket
Copy link
Contributor Author

anonymoussprocket commented Apr 7, 2021

limit 15, interval: 1200 will allow 15 threads (parallel requests) every 1.2 seconds or so. Data size wise, hen is at a point where a local cache makes sense for at least the token metadata.

@veqtor veqtor changed the base branch from master to develop April 7, 2021 06:19
@veqtor
Copy link
Collaborator

veqtor commented Apr 8, 2021

Hmm, this seems to be running of an older, possibly reverted version of master, if you could reimplement that would be good, also I don't understand what this PR solves

@veqtor
Copy link
Collaborator

veqtor commented Apr 13, 2021

Should I close this? We've reworked the code completely since this was made

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants