Skip to content

Conversation

@Shivs11
Copy link
Member

@Shivs11 Shivs11 commented Oct 16, 2025

READ BEFORE MERGING: All PRs require approval by both Server AND SDK teams before merging! This is why the number of required approvals is "2" and not "1"--two reviewers from the same team is NOT sufficient. If your PR is not approved by someone in BOTH teams, it may be summarily reverted.

What changed?

  • Added a monotonically increasing counter which shall be used by the history service when scheduling workflow/activity tasks.
  • Add an enum in the RoutingConfig which shall convey the routing config propagation status. This shall allow users to have a peek at the exact stage where an update, scheduled by a worker-versioning write API, has reached.

Why?

  • The main purpose of this is to resolve out-of-sync history and matching partitions when it comes to deciding how to route tasks

None, just new fields and enums, no new APIs

Server PR
temporalio/temporal#8570

@Shivs11 Shivs11 changed the title VersionCounter inside WorkflowExecutionVersioningInfo WorkerDeployments becoming async in nature. Oct 23, 2025
@Shivs11 Shivs11 marked this pull request as ready for review October 23, 2025 20:30
@Shivs11 Shivs11 requested review from a team as code owners October 23, 2025 20:30
@carlydf carlydf merged commit 060dbc3 into master Oct 31, 2025
4 checks passed
@carlydf carlydf deleted the ss/version_counter branch October 31, 2025 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants