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

Add specific worker backend notes and table with collected metrics for each backend #228

Merged
merged 6 commits into from
Nov 5, 2024

Conversation

carlosantoniodasilva
Copy link
Member

We needed a place to document a gotcha recently discovered with rescue-scheduler that wasn't loading our latency extension, so I added another sub-section under "Worker adapters" for that.

This also takes the opportunity to document what each adapter collects specifically, so it's easier to visualize where queue latency vs queue depth is available, and which can support busy (if enabled).

See the Readme live: https://github.com/judoscale/judoscale-ruby/tree/ca-readme?tab=readme-ov-file#specific-worker-backend-notes

Sample

Screenshot 2024-11-04 at 17 32 35

Screenshot 2024-11-04 at 17 32 39

So we can document a gotcha with using resque-scheduler and our
latency extension.
Makes it easier to visualize which supports queue latency vs depth vs
both, and busy support.
@carlosantoniodasilva carlosantoniodasilva self-assigned this Nov 4, 2024
@carlosantoniodasilva carlosantoniodasilva marked this pull request as ready for review November 4, 2024 20:41
Copy link
Collaborator

@adamlogic adamlogic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@carlosantoniodasilva carlosantoniodasilva merged commit 8c13482 into main Nov 5, 2024
@carlosantoniodasilva carlosantoniodasilva deleted the ca-readme branch November 5, 2024 12:46
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