Skip to content

Commit 600750a

Browse files
committed
Add blob and memo limits to NamespaceInfo (#670)
## Describe what has changed in this PR Add `blob_size_limit_error` and `memo_size_limit_error` to NamespaceInfo ## Tell your future self why have you made these changes `blob_size_limit_error` is the maximum size in bytes for payload fields in workflow history events (e.g., workflow/activity inputs and results, failure details, signal payloads). When exceeded, the server will reject the operation with an error. `memo_size_limit_error` is the maximum total memo size in bytes per workflow execution. Today, the size limit for input payloads is enforced on the server. SDK doesn't have a reliable way to know whether the request will be rejected on the server, because the size of the fields in the request exceeds namespace-level configured `blob_size_limit_error` and `memo_size_limit_error`. The motivation here, is for SDK to be able to do the check, failing the workflow task, allowing the user to fix the failure, rather than terminating the workflow on the server. ## Are there any breaking changes on binary or code level? No ## If this breaks the Server, please provide the Server PR to merge right after this PR was merged. N/A
1 parent 2fcd224 commit 600750a

File tree

4 files changed

+102
-31
lines changed

4 files changed

+102
-31
lines changed

namespace/v1/message.pb.go

Lines changed: 99 additions & 28 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

temporalproto/openapi/openapiv2.go

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

temporalproto/openapi/openapiv3.go

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)