micrometer-metrics/micrometer

Refinement/Clarification of LongTaskTimer when using Prometheus

Fermée

#6 507 ouverte le 3 juil. 2025

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)Java (1 133 forks)batch import
doc-updatehelp wantedregistry: prometheus

Métriques du dépôt

Stars
 (4 882 étoiles)
Métriques de merge PR
 (Merge moyen 1j 21h) (100 PRs mergées en 30 j)

Description

Please describe the feature request

Following a question in Slack with @shakuzen https://micrometer-metrics.slack.com/archives/C662HUJC9/p1751460825070189

The Migration to Prometheus 1.x client libs in micrometer 1.13. changed the behaviour for LongTaskTimer.

Specifically in said Slack thread it was concluded that:

  • The migration guide could be more precise in outlining two distinct changes
    • Histogram -> GaugeHistogram for LongTaskTimers with histogram buckets.
    • No more active or duration suffix. _active_count -> _count or _gcount, _duration_sum -> _sum or _gsum
  • The docs for LongTaskTimer or Micrometer Prometheus docs don't make it clear that .publishPercentileHistogram() might be necessary for certain use cases
  • Consider making all LongTaskTimers produce HistogramGauges and ditch Summary due to the semantic mismatch

Guide contributeur

Direction de recherche
Stack technique
Domaine
Type d'issue
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Prérequis
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.