micrometer-metrics/micrometer

Timer API does not support Kotlin suspend function (coroutines)

Open

#4.455 aberto em 7 de dez. de 2023

Ver no GitHub
 (4 comments) (6 reactions) (0 assignees)Java (935 forks)batch import
enhancementhelp wantedkotlin

Métricas do repositório

Stars
 (4.220 stars)
Métricas de merge de PR
 (Mesclagem média 1d 21h) (100 fundiu PRs em 30d)

Description

Please describe the feature request.

For standard (non suspend functions) it is possible to use record function to measure method invocation time:

Timer.builder("method.timed")
            .register(meterRegistry)
            .record { fn() } // fn() is a standard, non suspend function

However, it is not possible to call a suspend function from within record block:

Timer.builder("method.timed")
            .register(meterRegistry)
            .record { suspendFn() } // Suspension functions can be called only within coroutine body

The support for suspend functions should be added to the library.

Rationale Suspend functions are popular for non-blocking API, and it is currently not possible to measure time needed to call suspend functions without having a custom function to record the time.

Additional context Maybe this could be added as a extension function to the Timer interface.

Guia do colaborador

Direção de pesquisa
Implemente uma extensão de função suspensa para Timer.record() que aceite uma lambda suspensa, usando um wrapper baseado em corrotinas para medir o tempo de execução.
Pilha de tecnologia
javakotlin
Domain
observability
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
KotlinCoroutines
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70