Skip to content

Generate OTEL queue count metrics#47

Open
brandur wants to merge 1 commit intomasterfrom
brandur-queue-counts
Open

Generate OTEL queue count metrics#47
brandur wants to merge 1 commit intomasterfrom
brandur-queue-counts

Conversation

@brandur
Copy link
Copy Markdown
Contributor

@brandur brandur commented Apr 12, 2026

See [1] for context, but here we implement HookQueueStateCount in the
existing OTEL middleware so that it can emit metrics around queue
counts, a pretty handy feature that we have on definitive request for,
but which I'd guess most River users would like to have access to.

[1] riverqueue/river#1203

See [1] for context, but here we implement `HookQueueStateCount` in the
existing OTEL middleware so that it can emit metrics around queue
counts, a pretty handy feature that we have on definitive request for,
but which I'd guess most River users would like to have access to.

[1] riverqueue/river#1203
@brandur brandur force-pushed the brandur-queue-counts branch from c369df3 to dd91f10 Compare April 12, 2026 06:46
@brandur brandur requested a review from bgentry April 12, 2026 06:53
Copy link
Copy Markdown
Contributor

@bgentry bgentry left a comment

Choose a reason for hiding this comment

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

Should be fine once we work through the issues with #1203 👍

Comment on lines +43 to +51
replace github.com/riverqueue/river => ../../river

// replace github.com/riverqueue/river/riverdriver => ../../river/riverdriver
replace github.com/riverqueue/river/riverdriver => ../../river/riverdriver

// replace github.com/riverqueue/river/riverdriver => ../../river/riverdriver
replace github.com/riverqueue/river/riverdriver/riverpgxv5 => ../../river/riverdriver/riverpgxv5

// replace github.com/riverqueue/river/riverdriver/riverpgxv5 => ../../river/riverdriver/riverpgxv5
replace github.com/riverqueue/river/rivershared => ../../river/rivershared

// replace github.com/riverqueue/river/rivershared => ../../river/rivershared

// replace github.com/riverqueue/river/rivertype => ../../river/rivertype
replace github.com/riverqueue/river/rivertype => ../../river/rivertype
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not mergeable as-is, will need to point at a newer release or commit instead of using replace.

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