aws / aws/aws-durable-execution-sdk-python

revisit locking strategy in local runner

Offen
#639 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
pkg:testing
Vorherrschende Sprache
Python
Sterne
53
Forks
25
Ø Merge
1 T. 12 Std.
Gemergte PRs (30 T.)
37

Beschreibung

the extra `_state_lock` is actually not necessary, and if anything we could/should _probably_ remove it from get_navigable and friends, reason being the recent lane-based re-architecture isolated an execution into its own lane that serializes Checkpoint writes that run as CheckpointTask on that same lane, so the read and write cannot overlap.

however, that's wider refactor beyond the scope fo this PR, and since the lock is harmless and for the sake of symmetry might as well let it stand as introduced here and then revisit clean-up opportunities for redundant locks throughout later.

_Originally posted by @yaythomas in https://github.com/aws/aws-durable-execution-sdk-python/issues/637#issuecomment-5287117584_

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, den lokalen Runner und die get_navigable-Methoden zu lokalisieren, und verfolge dann, wie die lane-basierte Ausführung von CheckpointTask durchgeführte Checkpoint-Schreibvorgänge serialisiert. Ermittle, ob _state_lock für diese Lese- und Schreibvorgänge redundant ist. Als erledigt gilt die Aufgabe, wenn die Locking-Strategie nur dort vereinfacht wurde, wo dies sicher ist, und die Bereinigung redundanter Locks klar von dem umfassenderen Refactoring getrennt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
backend
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Ruhig
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
38/100

Neue Issues direkt in Ihr Postfach

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