temporalio / sdk-core

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

Ensure some rpcs use long-poll timeout #780

Closed Sushisource closed 1 month ago

Sushisource commented 1 month ago

What was changed

Added tests for grpc-timeout header (which was already working properly turns out) Made certain rpcs that do/can act like long-polls use the long poll timeout. One unrelated bit of cleanup around slot permit reservation I had pending.

Why?

Better timeouts

Checklist

  1. Closes https://github.com/temporalio/sdk-core/issues/774

  2. How was this tested: Added tests

  3. Any docs updates needed?