Skip to content

Add poller autoscaling namespace capability#723

Merged
Sushisource merged 2 commits intomasterfrom
poller-autoscale-capability
Mar 17, 2026
Merged

Add poller autoscaling namespace capability#723
Sushisource merged 2 commits intomasterfrom
poller-autoscale-capability

Conversation

@Sushisource
Copy link
Copy Markdown
Member

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?
title

Why?
Needed to allow SDKs to scale down on empty responses (otherwise, they can't be sure scale up could ever happen again, if the server is old)

Breaking changes
no

Server PR

@Sushisource Sushisource requested review from a team as code owners February 24, 2026 22:10
@Sushisource Sushisource enabled auto-merge (squash) March 17, 2026 19:59
@Sushisource Sushisource merged commit db5f593 into master Mar 17, 2026
4 checks passed
@Sushisource Sushisource deleted the poller-autoscale-capability branch March 17, 2026 20:01
02strich added a commit that referenced this pull request Mar 18, 2026
* origin/master:
  Add poller autoscaling namespace capability (#723)
  Add request header annotation and resource-id fields (#728)
  Trampolining: addition of fields to remove the infinite loops for pinned wfs  (#721)
  Add WorkerListInfo proto for efficient worker listing (#724)
  Add storage driver info to worker heartbeats (#722)
rkannan82 added a commit to temporalio/temporal that referenced this pull request Mar 19, 2026
## What changed?
Set the `PollerAutoscaling` namespace capability unconditionally to
`true`. This new capability was introduced in
temporalio/api#723.

## Why?
SDKs need to know the server supports poller scaling decisions so they
can safely scale down pollers on empty responses. Without this
capability, SDKs can't be sure the server would ever tell them to scale
back up (if the server is old).

Poller autoscaling is enabled/disabled on the worker (SDK) side — the
server always supports sending scaling decisions, so this capability is
hardcoded to `true`.

## How did you test it?
- [x] built
- [x] covered by existing tests (`TestCapabilitiesAndLimits`)

## Potential risks
None — additive capability flag, no behavioral change on the server.


Made with [Cursor](https://cursor.com)
stephanos pushed a commit to temporalio/temporal that referenced this pull request Mar 20, 2026
Set the `PollerAutoscaling` namespace capability unconditionally to
`true`. This new capability was introduced in
temporalio/api#723.

SDKs need to know the server supports poller scaling decisions so they
can safely scale down pollers on empty responses. Without this
capability, SDKs can't be sure the server would ever tell them to scale
back up (if the server is old).

Poller autoscaling is enabled/disabled on the worker (SDK) side — the
server always supports sending scaling decisions, so this capability is
hardcoded to `true`.

- [x] built
- [x] covered by existing tests (`TestCapabilitiesAndLimits`)

None — additive capability flag, no behavioral change on the server.

Made with [Cursor](https://cursor.com)
birme pushed a commit to eyevinn-osaas/temporal that referenced this pull request Mar 23, 2026
## What changed?
Set the `PollerAutoscaling` namespace capability unconditionally to
`true`. This new capability was introduced in
temporalio/api#723.

## Why?
SDKs need to know the server supports poller scaling decisions so they
can safely scale down pollers on empty responses. Without this
capability, SDKs can't be sure the server would ever tell them to scale
back up (if the server is old).

Poller autoscaling is enabled/disabled on the worker (SDK) side — the
server always supports sending scaling decisions, so this capability is
hardcoded to `true`.

## How did you test it?
- [x] built
- [x] covered by existing tests (`TestCapabilitiesAndLimits`)

## Potential risks
None — additive capability flag, no behavioral change on the server.


Made with [Cursor](https://cursor.com)
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.

3 participants