micrometer-metrics/micrometer

Remove @NullUnmarked workaround for NullAway bug on generic types

Open

#6.404 aberto em 14 de jun. de 2025

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)Java (935 forks)batch import
help wantednullabilitypolishtype: task

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

It seems <T extends @Nullable Object> does not work well in some situations, see https://github.com/uber/NullAway/issues/1075.

Because of this bug in NullAway, we need to use @NullUnmarked and "invert" the contract unnecessarily, see here and here.

Guia do colaborador

Direção de pesquisa
Examine o PR #6385 referenciado e as linhas específicas onde @NullUnmarked é usado (veja os links de diff na issue). Monitore a issue #1075 do NullAway para a correção upstream. Assim que o bug do NullAway for resolvido, remova as anotações @NullUnmarked e reverta o contrato invertido. Atualize quaisquer testes relevantes para garantir a correção sem a solução alternativa.
Pilha de tecnologia
java
Domain
build systemdeveloper experience
Tipo Issue
Bug
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.
Precisa de investigação
Prerequisites
JavaNullAway
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30