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: add some more debug logs #1979

Merged
merged 1 commit into from
Apr 25, 2024
Merged

chore: add some more debug logs #1979

merged 1 commit into from
Apr 25, 2024

Conversation

ellie
Copy link
Member

@ellie ellie commented Apr 25, 2024

P99 is usually <100ms which is excellent, but occasionally has big spikes to 1000ms. This is only on the record index.

I don't want this to get out of hand. I've ran a few test queries and they all complete very fast, and are purely index scans.

Hopefully this helps figure out if it's a specific user with tonnes of stores or something? Otherwise there could be something up with my db.

I should probably also figure out some proper log levels or tracing lol.

Checks

  • I am happy for maintainers to push small adjustments to this PR, to speed up the review cycle
  • I have checked that there are no existing pull requests for the same thing

P99 is usually <100ms which is excellent, but occasionally has big
spikes to 1000ms. This is only on the record index.

I don't want this to get out of hand. I've ran a few test queries and
they all complete very fast, and are purely index scans.

Hopefully this helps figure out if it's a specific user with tonnes of
stores or something? Otherwise there could be something up with my db.

I should probably also figure out some proper log levels or tracing lol.
@ellie ellie merged commit fc5522f into main Apr 25, 2024
17 checks passed
@ellie ellie deleted the ellie/v2-index-logs branch April 25, 2024 09:42
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.

1 participant