line/armeria

Consider to remove sorting at MeterIdPrefix

Chiusa

#2150 aperta il 3 ott 2019

Vedi su GitHubGuida contributor
 (10 commenti) (0 reazioni) (0 assegnatari)Java (863 fork)batch import
good first issueperformance

Metriche repository

Star
 (4552 stelle)
Metriche merge PR
 (Merge medio 5g 12h) (42 PR mergiate in 30 g)

Descrizione

Currently, we sort the micrometer tags at MeterIdPrefix to create Meter ID stably. https://github.com/line/armeria/blob/master/core/src/main/java/com/linecorp/armeria/common/metric/MeterIdPrefix.java#L95-L101

Since I saw this stacktrace from our service’s flamegraph, I would like to consider whether it is avoidable or not. Since it doesn’t have many samples, we can be ignored but because it is not optimal, it would be happy if we can avoid this sort.

Guida contributor

Direzione di ricerca
Tech stack
Dominio
Tipo issue
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Prerequisiti
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.