Skip to content

Sticky activity session options - #885

Draft
Shivs11 wants to merge 8 commits into
mainfrom
sticky-activity-session-options
Draft

Shivs11 wants to merge 8 commits into
mainfrom
sticky-activity-session-options

Conversation

@Shivs11

@Shivs11 Shivs11 commented Oct 20, 2026

Copy link
Copy Markdown
Member

What changed?

Why?

Breaking changes

Server PR

string stable_identity = 1;
}

message ActivitySessionOptions {

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

i envision these options to be sent on the command attributes when we are going to schedule an activity task -- if that is the case, i wonder if we should have something defined or thought of for the stable identity piece (single session isolation) given that we seem to have thought this in the poll request structures

if not, then i wonder if we do even need the single session options defined in the general WorkerSessionOptions

Comment on lines +17 to +19
oneof isolation {
SingleSessionOptions single_session = 1;
}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

should we also have a boolean for use_workflow_session here?

// workflow task is dispatched. The server persists that workflow ID and does
// not forward subsequent polls, including after workflow completion.
// Activity sticky polls are never forwarded to the normal queue.
message SingleSessionOptions {

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

thomas: says that maybe we should think about what this is called if we wanna ActivitySessionOptions

  • another thing that he says is that we may not require to think about introducing this right now given that this is experimental

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