Skip to content
This repository has been archived by the owner on Sep 26, 2024. It is now read-only.

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 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.

4 participants