micrometer-metrics/micrometer

Remove @NullUnmarked workaround for NullAway bug on generic types

Offen

#6.404 geöffnet am 14.06.2025

Auf GitHub ansehenContributor Guide
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (1.133 Forks)batch import
help wantednullabilitypolishtype: task

Repository-Metriken

Stars
 (4.882 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 1T 21h) (100 gemergte PRs in 30 T)

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

Research-Richtung
Untersuche den referenzierten PR #6385 und die spezifischen Zeilen, in denen @NullUnmarked verwendet wird (siehe Diff Links im Issue). Beobachte das NullAway Issue #1075 für die Upstream Korrektur. Sobald der NullAway Bug behoben ist, entferne die @NullUnmarked Annotationen und stelle den invertierten Vertrag wieder her. Aktualisiere relevante Tests, um die Korrektheit ohne den Workaround sicherzustellen.
Tech Stack
java
Domain
build systemdeveloper experience
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Recherche nötig
Voraussetzungen
JavaNullAway
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30