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

Repo per doc metrics #390

Merged
merged 2 commits into from
Oct 3, 2024
Merged

Repo per doc metrics #390

merged 2 commits into from
Oct 3, 2024

Conversation

alexjg
Copy link
Contributor

@alexjg alexjg commented Oct 1, 2024

Problem: when analyzing a sync server under heavy load it is difficult
to identify what is causing the load. There are lots of questions we
might have such as "what documents are in memory?" or "which documents are
taking longest in receiveSyncMessage?".

Solution: expose a "doc-metrics" event on Repo which is fired
whenever a documnt is loaded or receives a sync message. This can be
wired up to Prometheus or other monitoring tools to allow operators to
answer these questions.

@alexjg alexjg force-pushed the repo-per-doc-metrics branch from 9f082b1 to f36efc4 Compare October 1, 2024 16:36
alexjg added 2 commits October 2, 2024 23:41
Problem: when analyzing a sync server under heavy load it is difficult
to identify what is causing the load. There are lots of questions we
might have such as "what documents are in memory?" or "which documents are
taking longest  in `receiveSyncMessage`?". 

Solution: expose a `"doc-metrics"` event on `Repo` which is fired
whenever a documnt is loaded or receives a sync message. This can be
wired up to Prometheus or other monitoring tools to allow operators to
answer these questions.

Add num ops and changes to events
@alexjg alexjg force-pushed the repo-per-doc-metrics branch from f36efc4 to fdcee17 Compare October 2, 2024 22:42
@alexjg alexjg merged commit 77451ec into main Oct 3, 2024
2 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.

1 participant