Skip to content

docs(config): say what cache_size zero actually does - #811

Closed
xepozz wants to merge 1 commit into
temporalio:masterfrom
xepozz:docs/cache-size-zero-means-default
Closed

docs(config): say what cache_size zero actually does#811
xepozz wants to merge 1 commit into
temporalio:masterfrom
xepozz:docs/cache-size-zero-means-default

Conversation

@xepozz

@xepozz xepozz commented Aug 23, 2026

Copy link
Copy Markdown
Collaborator

Reason for This PR

[Author TODO: add issue # or explain reasoning.]

Description of Changes

InitDefaults replaces a zero cache_size with 10000, so zero cannot be used to turn the sticky cache off - only a negative value does that.

License Acceptance

By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.

PR Checklist

[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]

  • All commits in this PR are signed (git commit -s).
  • The reason for this PR is clearly provided (issue no. or explanation).
  • The description of changes is clear and encompassing.
  • Any required documentation changes (code and docs) are included in this PR.
  • Any user-facing changes are mentioned in CHANGELOG.md.
  • All added/changed functionality is tested.

InitDefaults replaces a zero cache_size with 10000, so zero cannot be used to turn the
sticky cache off - only a negative value does that.
@xepozz
xepozz requested a review from rustatian as a code owner August 23, 2026 09:33
Comment thread schema.json
"minLength": 1
},
"cache_size": {
"description": "Sticky cache size. Sticky workflow execution is the affinity between workflow tasks of a specific workflow execution to a specific worker. The benefit of sticky execution is that the workflow does not have to reconstruct state by replaying history from the beginning. The cache is shared between workers running within same process. This must be called before any worker is started. If not called, the default size of 10K (which may change) will be used.",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why was the comment with This must be .. removed?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

So if you call config parameters I'll back it then, usually I don't

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Not sure I get you; could you please rephrase?

@xepozz xepozz closed this Aug 31, 2026
@xepozz
xepozz deleted the docs/cache-size-zero-means-default branch August 31, 2026 15:25
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.

2 participants