temporalio / temporalio/sdk-rust

[Feature Request] Move more of the `patched()` logic to Core rather than langs

Open
#869 0 comments 0 reactions 1 assignee View on GitHub

@THardy98 is already working on this.

Since Mar 25, 2025.

enhancement
Dominant language
Rust
Stars
523
Forks
161
Avg merge
1d 7h
Merged PRs (30d)
67

Description

Describe the solution you'd like

In short:

  • The patched() API now needs support explicit memoization support.
  • Logic for non-memoization vs memoization needs to be handled in Core rather than in each lang.
  • Make sure Core's non-memoized logic stays compatible with TS SDK's current implementation, and that the memoized logic stays compatible with Python and .Net.

See temporalio/features#591 for details.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.