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.

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

Reduce the cache ttl for public donations and total money collected. #572

Merged
merged 1 commit into from
Nov 2, 2023
Merged

Conversation

slavcho
Copy link
Contributor

@slavcho slavcho commented Nov 1, 2023

The idea of the cache is to help in extreme scenarios when many requests are being fired.
One request every 2 seconds should be easy to handle by the backend.

Motivation and context

Many people want to see their donation immediately after the payment.
With the 30 seconds cache - this was impossible and confusing.

Testing

Fired multiple requests and observed in the log that the actual db search is executed only once every 2 seconds for the public donations.

The idea of the cache is to help in extreme scenarios when many requests are being fired.
One request every 2 seconds should be easy to handle by the backend.
@slavcho slavcho added the run tests Allows running the tests workflows for forked repos label Nov 1, 2023
@github-actions github-actions bot removed the run tests Allows running the tests workflows for forked repos label Nov 1, 2023
Copy link

github-actions bot commented Nov 1, 2023

✅ Tests will run for this PR. Once they succeed it can be merged.

1 similar comment
Copy link

github-actions bot commented Nov 1, 2023

✅ Tests will run for this PR. Once they succeed it can be merged.

@igoychev igoychev merged commit d385420 into podkrepi-bg:master Nov 2, 2023
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants