Skip to content

Commit 7f7a3d6

Browse files
committed
Worker Heartbeat: Use Worker Grouping instead of Runtime for Key in nexus TQ (#637)
_**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._ <!-- Describe what has changed in this PR --> **What changed?** use `worker_grouping_key` instead of `runtime_key` for worker command nexus task queue <!-- Tell your future self why have you made these changes --> **Why?** There can be multiple clients (i.e. with different auths) within the same runtime/namespace, so we want a more specific category for these nexus worker command requests <!-- Are there any breaking changes on binary or code level? --> **Breaking changes** No, WIP feature <!-- If this breaks the Server, please provide the Server PR to merge right after this PR was merged. --> **Server PR**
1 parent d7a5a2a commit 7f7a3d6

File tree

4 files changed

+12
-15
lines changed

4 files changed

+12
-15
lines changed

temporalproto/openapi/openapiv2.go

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

temporalproto/openapi/openapiv3.go

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

worker/v1/message.pb.go

Lines changed: 9 additions & 12 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)