micrometer-metrics/micrometer

Remove @NullUnmarked workaround for NullAway bug on generic types

Open

#6.404 geöffnet am 14. Juni 2025

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (4.220 Stars) (935 Forks)batch import
help wantednullabilitypolishtype: task

Beschreibung

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.

Contributor Guide

Tech Stack
java
Domain
tooling
Issue Type
refactor
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
blocked
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
JavaNullAwayGenerics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
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.