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

Backport per-handler metrics to v23.1 #13344

Merged
merged 16 commits into from
Sep 11, 2023

Conversation

StephanDollberg
Copy link
Member

@StephanDollberg StephanDollberg commented Sep 8, 2023

Backport of PR #10623
Backport of PR #12036
Backport of PR #12295
Backport of PR #12460

Backports all PRs/commits related to per handler metrics.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v23.2.x
  • v23.1.x
  • v22.3.x

@StephanDollberg StephanDollberg changed the title Stephan/handler backport v231 Backport per-handler metrics to v23.1 Sep 8, 2023
(cherry picked from commit afdcdc2)
Since `request_tracker` will count the completion of the request when
it is destroyed this removed line will double count failed completed
requests.

(cherry picked from commit 1c222ca)
This histogram type avoids using `hdr_hist` to generate seastar
histograms in metrics. It instead implements a simple powers of
two histogram that reduces memory utilization from kilobytes to
`number_of_buckets` * sizeof(uint64_t).

(cherry picked from commit b1ff1b7)
…om log_hist_internal to a public seastar hist

Co-authored-by: Ben Pope <[email protected]>
(cherry picked from commit 6b578bd)
This reduces the memory required to store a histogram from kilobytes to
208 bytes. It also speeds up recording to the histogram by 2x.

(cherry picked from commit c2959f5)
@StephanDollberg StephanDollberg marked this pull request as ready for review September 8, 2023 19:23
@piyushredpanda piyushredpanda merged commit e14de94 into v23.1.x Sep 11, 2023
12 checks passed
@piyushredpanda piyushredpanda deleted the stephan/handler-backport-v231 branch September 11, 2023 22:11
@BenPope BenPope added this to the v23.1.18 milestone Oct 10, 2023
@BenPope BenPope added the kind/backport PRs targeting a stable branch label Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/redpanda kind/backport PRs targeting a stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants