Skip to content

Conversation

@sophiatev
Copy link
Contributor

@sophiatev sophiatev commented Nov 20, 2025

This PR adds a new properties field to the EntityBatchRequest, which is a dictionary that contains information about extended sessions (for example, whether or not they are enabled, the extended session idle timeout in seconds, and whether or not entity state was attached to the request). It also adds a requiresState field to the EntityBatchResponse which is used by the worker to let the host know that it needs an entity state but none was attached to the request (maybe the cache entry for that entity expired, for example).

Copy link
Member

@cgillum cgillum left a comment

Choose a reason for hiding this comment

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

LGTM - I suggest we wait to merge this until the implementations are signed off, just in case we discover that more changes are needed.

@sophiatev sophiatev merged commit b7e260a into main Dec 29, 2025
1 check passed
@sophiatev sophiatev deleted the stevosyan/extended-sessions-for-entities-isolated branch December 29, 2025 22:03
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