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

feat(gateway-engine-core): thread-safe batched non-blocking metrics consumer #2126

Merged
merged 1 commit into from
Jun 24, 2022

Conversation

msavy
Copy link
Member

@msavy msavy commented Jun 24, 2022

A generic batched metrics consumer that should allow batched, thread-safe non-blocking metrics
ingestion by many different metrics implementations, rather than reinventing this pattern many
times.

This implementation runs as fast as it can, using a spin-wait worker thread that polls a
non-blocking queue. It passes these metrics to a call-handler which may then go and perform
actions such as serializing and transmitting those metrics.

This implementation does not have a minimum batch size nor minimum polling interval.

A generic batched metrics consumer that should allow batched, thread-safe non-blocking metrics
ingestion by many different metrics implementations, rather than reinventing this pattern many
times.

This implementation runs as fast as it can, using a spin-wait worker thread that polls a
non-blocking queue. It passes these metrics to a call-handler which may then go and perform
actions such as serializing and transmitting those metrics.

This implementation does not have a minimum batch size nor minimum polling interval.
@msavy msavy changed the title feat(gateway-engine-core): batched metrics consumer feat(gateway-engine-core): thread-safe batched non-blocking metrics consumer Jun 24, 2022
@msavy msavy merged commit 1bd43dc into master Jun 24, 2022
@msavy msavy deleted the metrics-consumer branch June 24, 2022 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant