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

distributor(ha_tracker): add native histogram for electedReplicaPropagationTime #10264

Conversation

NickAnge
Copy link
Contributor

@NickAnge NickAnge commented Dec 17, 2024

What this PR does

This PR introduces native histogram support for electedReplicaPropagationTime in order to get better granularity and visibility on the tail latencies above 10s during the testing phase of memberlist as KvStore for ha_tracker

Which issue(s) this PR fixes or relates to

Relates to [ha_tracker][memberlist] Testing Phase

Checklist

  • Tests updated.
  • Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • about-versioning.md updated with experimental features.

@NickAnge NickAnge marked this pull request as ready for review December 17, 2024 13:30
@NickAnge NickAnge requested a review from a team as a code owner December 17, 2024 13:30
@NickAnge NickAnge marked this pull request as draft December 17, 2024 13:30
@NickAnge NickAnge changed the title distributor: add native histogram support for electedReplicaPropagati… distributor(ha_tracker): add native histogram for electedReplicaPropagationTime Dec 17, 2024
@NickAnge NickAnge marked this pull request as ready for review December 17, 2024 14:54
CHANGELOG.md Outdated Show resolved Hide resolved
@NickAnge NickAnge force-pushed the nickange/ha_tracker/change-propagation-time-to-native-histogram branch from 267a696 to f00690d Compare December 17, 2024 16:27
@dimitarvdimitrov dimitarvdimitrov merged commit cb24dfe into main Dec 18, 2024
29 checks passed
@dimitarvdimitrov dimitarvdimitrov deleted the nickange/ha_tracker/change-propagation-time-to-native-histogram branch December 18, 2024 09:49
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