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(perf) Add latency metrics to the messages coming from the commit log #38

Merged
merged 2 commits into from
Feb 8, 2022

Conversation

fpacifici
Copy link
Contributor

Lately we observed weird temporary increases in subscriptions consumer latency on selected partitions.
It is hard to understand exactly who is causing them.
One theory is that somehow the commit log messages could be delayed or could be processed too late by the subscriptions consumer.

Adding latency metrics to the code that consumes the commit log to rule out this hypothesis.
Specifically we record both the difference between the current time and the broker time the commit log message was produced at and the difference between the current time and the broker timestamp of the last message the commit log messages refers to.
The first gives us an indication if the commit log messages are late.
The second gives us an indication if the main consumer is consuming old messages.

@fpacifici fpacifici requested a review from a team February 8, 2022 01:22
@fpacifici fpacifici merged commit 7a9d86d into main Feb 8, 2022
@fpacifici fpacifici deleted the feat/add_commit_metrics branch February 8, 2022 17:05
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