Skip to content

expose state size bytes when describing standalone activities#9962

Merged
fretz12 merged 1 commit intotemporalio:mainfrom
fretz12:fredtzeng/state-size-bytes
Apr 15, 2026
Merged

expose state size bytes when describing standalone activities#9962
fretz12 merged 1 commit intotemporalio:mainfrom
fretz12:fredtzeng/state-size-bytes

Conversation

@fretz12
Copy link
Copy Markdown
Contributor

@fretz12 fretz12 commented Apr 15, 2026

What changed?

Populated the StateSizeBytes field in buildActivityExecutionInfo from executionInfo.ApproximateStateSize, and added assertions for it across the standalone activity integration tests

Why?

Filling it in gives API consumers visibility into activity state size, which is useful for monitoring resource usage and billing purposes.

How did you test it?

  • built
  • run locally and tested manually
  • covered by existing tests
  • added new unit test(s)
  • added new functional test(s)

@fretz12 fretz12 marked this pull request as ready for review April 15, 2026 15:53
@fretz12 fretz12 requested review from a team as code owners April 15, 2026 15:53
Copy link
Copy Markdown
Contributor

@dandavison dandavison left a comment

Choose a reason for hiding this comment

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

LGTM!

@fretz12 fretz12 merged commit 2a2a5f2 into temporalio:main Apr 15, 2026
46 checks passed
@fretz12 fretz12 deleted the fredtzeng/state-size-bytes branch April 15, 2026 18:01
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