search

temporalio / sdk-python

Temporal Python SDK
MIT License
472 stars 75 forks source link

[Bug] Update requests should set first_execution_run_id #682

Open dandavison opened 3 weeks ago

dandavison commented 3 weeks ago

Update requests do not send first_execution_run_id: https://github.com/temporalio/sdk-python/blob/0b327b04a95fc7954ed2c9e4e1a6c5de47c51ebd/temporalio/client.py#L5252-L5285

However, they should, and other SDKs do.

Also, is it correct that we don't use the run ID from the response when constructing the returned update handle?