Improve thread safety docs for fields in ServerImpl
Offen
Type: Documentation
- Vorherrschende Sprache
- Java
- Sterne
- 12.1k
- Forks
- 4k
- Ø Merge
- 2 T. 17 Std.
- Gemergte PRs (30 T.)
- 37
Beschreibung
Split out of https://github.com/grpc/grpc-java/pull/2032#discussion_r70340687. `usingSharedExecutor` and `executor` aren't using `GuardedBy` semantics. It isn't clear to @carl-mastrangelo that the current code is valid. Improve the docs/code to correct/make the correctness more clear.
Beitragsleitfaden
Rechercherichtung
Beginne mit ServerImpl und prüfe die Thread-Sicherheitsbehandlung der Felder usingSharedExecutor und executor. Lies die im Issue verlinkte Diskussion zur Frage der GuardedBy-Semantik und präzisiere anschließend die Dokumentation oder den Code, sodass die bestehende Korrektheit eindeutig ist. Erledigt ist die Aufgabe, wenn die Synchronisierungserwartungen für beide Felder klar sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- backend
- Issue-Typ
- Dokumentation
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 30/100