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

chore: change interval to 30 seconds rather than every second #1236

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

roshaans
Copy link

@roshaans roshaans commented Sep 14, 2023

According to the Database Query Insights, this query was run in the backend over 20 thousand times in just the past hour alone (Almost 4 million requests in the past 7 days) and each request takes on average 1.8 seconds to make and the result is just always one row.

This PR hopes to lessen this bottleneck query by extending the interval for this query from every second to every 30 seconds.

@roshaans roshaans temporarily deployed to dev-backend-testnet September 14, 2023 14:55 — with GitHub Actions Inactive
@roshaans roshaans temporarily deployed to dev-backend-mainnet September 14, 2023 14:55 — with GitHub Actions Inactive
@roshaans roshaans temporarily deployed to dev-frontend September 14, 2023 14:57 — with GitHub Actions Inactive
@roshaans roshaans enabled auto-merge (squash) September 14, 2023 20:46
@roshaans
Copy link
Author

Playwright tests seem to be failing because of a dependency on the database.

Screenshot 2023-09-18 at 5 45 21 PM

@frol
Copy link
Collaborator

frol commented Sep 19, 2023

@roshaans Unit-tests fail for a completely different reason (snapshot mismatch), I don't understand how they used to pass, though 🤷‍♂️

@rtsainear rtsainear temporarily deployed to dev-backend-mainnet October 5, 2023 22:17 — with GitHub Actions Inactive
@rtsainear rtsainear temporarily deployed to dev-backend-testnet October 5, 2023 22:17 — with GitHub Actions Inactive
@rtsainear rtsainear temporarily deployed to dev-frontend October 5, 2023 22:19 — with GitHub Actions Inactive
@rtsainear rtsainear temporarily deployed to dev-backend-testnet October 6, 2023 03:36 — with GitHub Actions Inactive
@rtsainear rtsainear temporarily deployed to dev-backend-mainnet October 6, 2023 03:36 — with GitHub Actions Inactive
@rtsainear rtsainear temporarily deployed to dev-frontend October 6, 2023 03:38 — with GitHub Actions Inactive
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.

None yet

4 participants