Skip to content

Conversation

@xwduan
Copy link
Contributor

@xwduan xwduan commented Dec 10, 2025

What changed?

Trim SyncWorkflowState request branch token and log request size when error

Why?

  1. To reduce the request size
  2. help on debug when payload size is greater than grpc max msg size

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)

Potential risks

No risk.

@xwduan xwduan changed the title Trim request branch token and log request size when error Trim SyncWorkflowState request branch token and log request size when error Dec 10, 2025
@xwduan xwduan marked this pull request as ready for review December 10, 2025 22:22
@xwduan xwduan requested review from a team as code owners December 10, 2025 22:22
@xwduan xwduan enabled auto-merge (squash) December 11, 2025 06:11
@xwduan xwduan merged commit a3a5326 into main Dec 12, 2025
58 checks passed
@xwduan xwduan deleted the will/add_log_for_sync_workflow_state branch December 12, 2025 01:35
stpierre pushed a commit that referenced this pull request Dec 12, 2025
… error (#8794)

## What changed?
Trim SyncWorkflowState request branch token and log request size when
error

## Why?
1. To reduce the request size
2. help on debug when payload size is greater than grpc max msg size

## How did you test it?
- [x] built
- [x] run locally and tested manually
- [x] covered by existing tests
- [ ] added new unit test(s)
- [ ] added new functional test(s)

## Potential risks
No risk.
@temporal-cicd temporal-cicd bot mentioned this pull request Dec 12, 2025
temporal-cicd bot pushed a commit that referenced this pull request Dec 12, 2025
… error (#8794)

## What changed?
Trim SyncWorkflowState request branch token and log request size when
error

## Why?
1. To reduce the request size
2. help on debug when payload size is greater than grpc max msg size

## How did you test it?
- [x] built
- [x] run locally and tested manually
- [x] covered by existing tests
- [ ] added new unit test(s)
- [ ] added new functional test(s)

## Potential risks
No risk.
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