micrometer-metrics/micrometer

Remove @NullUnmarked workaround for NullAway bug on generic types

Open

#6 404 ouverte le 14 juin 2025

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)Java (4 220 stars) (935 forks)batch import
help wantednullabilitypolishtype: task

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.

Guide contributeur

Stack technique
java
Domaine
tooling
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
blocked
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
JavaNullAwayGenerics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Examine the referenced PR #6385 and the specific lines where @NullUnmarked is used (see diff links in issue). Monitor the NullAway issue #1075 for the upstream fix. Once the NullAway bug is resolved, remove the @NullUnmarked annotations and revert the inverted contract. Update any relevant tests to ensure correctness without the workaround.