ReCounters.java returns a counter without retries from create()

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#149 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
1/5
Geschätzter Aufwand
Unter einer Stunde
Anfängerfreundlichkeit
85/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
java
Bereich
backend

Rechercherichtung

Die Änderung gehört in create in src/main/java/co/stateful/retry/ReCounters.java, das derzeit this.origin.create(name) unverpackt zurückgibt. Vergleiche es mit get, das sein Ergebnis bereits in ReCounter verpackt, und verpacke das Ergebnis von create auf dieselbe Weise. Fertig ist es, wenn set und incrementAndGet auf einem neu erstellten Zähler bei IOException erneut versuchen, so wie Zähler aus get es tun. Die Vorgabe nennt keine Testdatei, also prüfe, ob es bereits einen Retry-Test gibt, den du erweitern kannst.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

ReCounters.create("hits") returns the original counter without a ReCounter wrapper. Consequently, new ReSttc(sttc).counters().create("hits").set(10L) propagates the first IOException from set, even if a second attempt would succeed. The same counter obtained through get("hits") has retry protection. This contradicts ReSttc's promise to retry requests and also affects incrementAndGet on a newly created counter.

In src/main/java/co/stateful/retry/ReCounters.java, lines 58–60:

public Counter create(final String name) throws IOException {
    return this.origin.create(name);
}

The annotation on create retries creation itself; it cannot intercept later method calls on the returned original counter. get already wraps its result in ReCounter on line 79, whose set and incrementAndGet methods carry @RetryOnFailure.

Wrap the result of create in the same decorator:

public Counter create(final String name) throws IOException {
    return new ReCounter(this.origin.create(name));
}

This one-line change gives counters returned by both entry points the same retry behavior.

Vorherrschende Sprache
Java
Sterne
10
Forks
3
Ø Merge
4 Std. 22 Min.
Gemergte PRs (30 T.)
9

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus sttc/java-sdk

Alle Issues in sttc/java-sdk

Ähnliche Issues

Weitere Issues zu Java

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.