Skip to content

Set poller autoscaling namespace capability#9572

Merged
rkannan82 merged 4 commits intomainfrom
kannan/poller-autoscaling-capability
Mar 19, 2026
Merged

Set poller autoscaling namespace capability#9572
rkannan82 merged 4 commits intomainfrom
kannan/poller-autoscaling-capability

Conversation

@rkannan82
Copy link
Copy Markdown
Contributor

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?

  • built
  • covered by existing tests (TestCapabilitiesAndLimits)

Potential risks

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

Made with Cursor

Set the PollerAutoscaling namespace capability unconditionally to true,
so SDKs know the server supports poller scaling decisions and can safely
scale down pollers on empty responses.

Made-with: Cursor
@rkannan82 rkannan82 force-pushed the kannan/poller-autoscaling-capability branch from a6d6837 to d2d3be0 Compare March 18, 2026 21:34
Comment thread service/frontend/namespace_handler.go
@rkannan82 rkannan82 requested a review from Sushisource March 18, 2026 23:00
@rkannan82 rkannan82 marked this pull request as ready for review March 18, 2026 23:00
@rkannan82 rkannan82 requested review from a team as code owners March 18, 2026 23:00
@rkannan82 rkannan82 force-pushed the kannan/poller-autoscaling-capability branch from 6229a8e to f34a863 Compare March 18, 2026 23:13
@rkannan82 rkannan82 force-pushed the kannan/poller-autoscaling-capability branch from f34a863 to f38867c Compare March 18, 2026 23:13
@rkannan82 rkannan82 requested a review from ShahabT March 18, 2026 23:14
@rkannan82 rkannan82 enabled auto-merge (squash) March 18, 2026 23:27
@rkannan82 rkannan82 merged commit 1bf6950 into main Mar 19, 2026
46 checks passed
@rkannan82 rkannan82 deleted the kannan/poller-autoscaling-capability branch March 19, 2026 21:22
stephanos pushed a commit 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