Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Reduce amount of traffic to coingecko backend? #228

Open
lyricnz opened this issue Oct 14, 2021 · 3 comments
Open

Reduce amount of traffic to coingecko backend? #228

lyricnz opened this issue Oct 14, 2021 · 3 comments
Labels
enhancement New feature or request

Comments

@lyricnz
Copy link
Collaborator

lyricnz commented Oct 14, 2021

We should review all the requests made to the backend, and see if we can reduce this:

  • load infrequently used data on demand
  • load "skeleton" data and flesh it out on demand
  • refresh chart data at a rate sensitive to the selected chart range (no point in refreshing yearly chart data every few seconds)
  • review polling frequency
  • find an API alternative to GlobalCharts() data rather than hitting the website endpoint see no enough data for chart #226
  • consider Cloudflare-defeating HTTP client (very uncool)
@lyricnz lyricnz added the enhancement New feature or request label Oct 14, 2021
@lyricnz
Copy link
Collaborator Author

lyricnz commented Oct 21, 2021

Duplicate requests to GET https://api.coingecko.com/api/v3/global

@lyricnz lyricnz mentioned this issue Oct 24, 2021
Closed
@lyricnz
Copy link
Collaborator Author

lyricnz commented Oct 24, 2021

Review cache duration and effectiveness (add metrics)

lyricnz added a commit to lyricnz/cointop that referenced this issue Oct 25, 2021
@lyricnz
Reduce the number of pages to 10 (1000 coins) to reduce the load on b…
21174a5 
…ackend, and increase refresh time. See cointop-sh#104 cointop-sh#228
@lyricnz lyricnz mentioned this issue Oct 25, 2021
Merged
lyricnz added a commit that referenced this issue Oct 28, 2021
@lyricnz
Reduce the number of pages to 10 (1000 coins) to reduce the load on b…
ac946a7 
…ackend... (#255)

* Reduce the number of pages to 10 (1000 coins) to reduce the load on backend, and increase refresh time. See #104 #228

* Add note about --max-pages and --per-page
This was referenced Nov 8, 2021
Open
Open
@lyricnz
Copy link
Collaborator Author

lyricnz commented Nov 14, 2021

Related : #276 support coins very far down the list

Refreshing all the coins, around in a loop, doesn't feel like a sustainable strategy

@lyricnz lyricnz mentioned this issue Nov 14, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lyricnz