aws / aws/aws-durable-execution-sdk-python
revisit locking strategy in local runner
- 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
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