temporalio / sdk-core

Core Temporal SDK that can be used as a base for language specific Temporal SDKs
MIT License
278 stars 77 forks source link

Local timeouts for remote activities #620

Closed Sushisource closed 1 year ago

Sushisource commented 1 year ago

What was changed

Core now will send cancellations if an activity would have already timed out according to server but we haven't learned of that (typically, because it isn't heartbeating). There is a configurable buffer defaulting to 5s, to avoid races with server (we would prefer to learn from server).

Why?

Nice to have your activities get cancelled when they would have been even if they don't heartbeat

Checklist

  1. Closes https://github.com/temporalio/features/issues/170

  2. How was this tested: Unit & Integ

  3. Any docs updates needed?