Skip to content

Clarify namespace poller group snapshots - #871

Open
yuandrew wants to merge 1 commit into
mainfrom
fix-poller-groups-docs
Open

yuandrew wants to merge 1 commit into
mainfrom
fix-poller-groups-docs

Conversation

@yuandrew

Copy link
Copy Markdown
Contributor

What changed?
Make it clear that poller groups are namespace wide, not task queue unique.

Why?
Codex got confused (and I also forgot), i think we can make this more explicitly clear in commends

Breaking changes
No, comment only change

Server PR

@yuandrew
yuandrew requested review from a team September 17, 2026 10:03
@yuandrew
yuandrew enabled auto-merge (squash) September 17, 2026 23:21

@ShahabT ShahabT left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we should mark the info Poll APIs as NS level. The one returned by the DescribeNamespace sure is NS level, as the seed value for all TQs in the NS.

True that as of today the weights are all the same for all task queues. But SDK should not assume so. We are adding backlog awareness inside each cell's partition selections. Next natural step is to extend that to make MCN weights backlog-aware. Once that is done, the weights for each TQ would change.

Also note that things like cell-pinning would change the set of cells for a given TQ.

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.

4 participants