temporalio / temporalio/sdk-python

[Feature Request] Add explicit memoization support to the `patched()` API

Offen
#754 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

enhancement
Vorherrschende Sprache
Python
Sterne
1.2k
Forks
241
Ø Merge
3 T. 21 Std.
Gemergte PRs (30 T.)
55

Beschreibung

Describe the solution you'd like

In short:

  • Add a memoized argument to the patched and deprecate_patch APIs.

  • Keep backward compatibility on those functions for non-specifying the memoized argument, but deprecate those signatures. Make sure documentation clearly indicate how existing calls may safely get converted.

  • Delegate handling of the patched logic to Core.

See temporalio/features#591 for details.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginnen Sie mit den API-Einstiegspunkten patched und deprecate_patch und lesen Sie anschließend temporalio/features#591 für die detaillierten Anforderungen. Die Arbeit ist abgeschlossen, wenn beide APIs explizites Verhalten von memoized akzeptieren, die bestehenden Signaturen sicher beibehalten und als veraltet markieren, Migrationshinweise dokumentieren und die Behandlung von patched an Core delegieren.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
api, backend
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

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