✅ Activity Heartbeat Throttling

[Spikhalskiy]

Align the behavior of activity heartbeat throttling on all SDKs.

Internal Temporal design doc: https://www.notion.so/temporalio/Activity-heartbeats-inconsistent-behavior-between-SDKs-32d81da0748044a8932481d2c9864791 Exported: Activity_heartbeats_inconsistent_behavior_between_SDKs.pdf

Design

Add 2 new WorkerOptions:

• maxHeartbeatThrottleInterval - default 60s
• defaultHeartbeatThrottleInterval - default 30s

throttleInterval = min(heartbeat_timeout * 0.8 if heartbeat_timeout is provided, else defaultHeartbeatThrottleInterval, maxHeartbeatThrottleInterval)

Rationale

• Make the behavior explicit and configurable
• Keep the behavior backwards compatible
  • Java currently has an implicit maximum (30s) and default (0)
  • Go has an implicit default (10m) and no maximum
  • Core is not yet stable, from user feedback the behavior is confusing
• Not heartbeating means no cancellations, need a default value
• Consistency between the SDKs

Tasks

https://github.com/temporalio/sdk-go/issues/656 https://github.com/temporalio/sdk-java/issues/906

[bergundy]

This was already implemented in all SDKs. Leaving it open to add a feature in this repo.