Skip to content

feat: warn about oversized payloads - #811

Open
xepozz wants to merge 9 commits into
temporalio:masterfrom
xepozz:feature/payload-warnings
Open

xepozz wants to merge 9 commits into
temporalio:masterfrom
xepozz:feature/payload-warnings

Conversation

@xepozz

@xepozz xepozz commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

What was changed

A [TMPRL1103] warning is logged when an outgoing gRPC request or a Workflow command carries payloads above the configured limit (512 KiB payloads, 2 KiB memo by default).

Why?

Closes #802.

Checklist

  • Covered by tests

Log a [TMPRL1103] warning when an outgoing gRPC request or a Workflow
command carries payloads above the configured limit.
RoadRunner already emits TMPRL1103 for the commands it sends: sdk-go installs
the payload limit visitor in NewAggregatedWorker with the same 512 KiB / 2 KiB
defaults. The PHP side was never wired into WorkerFactory, and wiring it would
convert every payload twice.
… the second Client

Also measure the termination details of a batch operation.
@xepozz
xepozz marked this pull request as ready for review September 17, 2026 07:36
@xepozz
xepozz requested review from a team, roxblnfk and wolfy-j as code owners September 17, 2026 07:36
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.

Warn if the SDK tried to send a payload above a specific size - PHP

1 participant