temporalio / temporalio/sdk-python
[Feature Request] Add explicit memoization support to the `patched()` API
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 1.2k
- Forks
- 241
- Avg merge
- 3d 21h
- Merged PRs (30d)
- 55
Description
Describe the solution you'd like
In short:
-
Add a
memoizedargument to thepatchedanddeprecate_patchAPIs. -
Keep backward compatibility on those functions for non-specifying the
memoizedargument, but deprecate those signatures. Make sure documentation clearly indicate how existing calls may safely get converted. -
Delegate handling of the
patchedlogic to Core.
See temporalio/features#591 for details.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the patched and deprecate_patch API entry points, then read temporalio/features#591 for the detailed requirements. The work is done when both APIs accept explicit memoized behavior, preserve and deprecate the existing signatures safely, document migration guidance, and delegate patched handling to Core.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- api, backend
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100