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

[Bug] UpdateRandomSeed jobs should go first with patches #790

Closed Sushisource closed 3 weeks ago

Sushisource commented 1 month ago

These can, like patches, affect the logic of anything that happens subsequently and hence they should go first.

It's not immediately clear if this isn't already implicitly the case. If it is, we don't need a flag. If it isn't, we do. So, it's worth proving with tests/analysis if we can avoid introducing a flag. Initial impl w/o the tests here: https://github.com/temporalio/sdk-core/pull/789