Skip to content

fix(grpc): Increase max message size to 10MB#658

Merged
untitaker merged 1 commit into
mainfrom
fix-grpc-max-message-size
May 29, 2026
Merged

fix(grpc): Increase max message size to 10MB#658
untitaker merged 1 commit into
mainfrom
fix-grpc-max-message-size

Conversation

@untitaker
Copy link
Copy Markdown
Member

@untitaker untitaker commented May 29, 2026

Profiles sometimes take 7 MB, and a lot of our default kafka max message sizes are 10 MB. So I think 10 MB is a tolerable default.

Summary

  • Increase gRPC max message size from default 4MB to 10MB on both broker and worker
  • Broker: Add grpc_max_message_size config option, use it when creating WorkerServiceClient
  • Worker: Add TASKBROKER_GRPC_MAX_MESSAGE_SIZE env var, apply to gRPC server options

Test plan

  • Rust tests pass (cargo test)
  • Python tests pass (pre-existing failures unrelated)
  • Deploy and verify stuck tasks in s4s2 ingest-profiles-raw get processed

ref INC-2209

🤖 Generated with Claude Code

@untitaker untitaker requested a review from a team as a code owner May 29, 2026 20:32
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 29, 2026

INC-2209

The default gRPC max message size is 4MB. Tasks with payloads larger than
this fail to be pushed to workers and get stuck in an infinite retry loop,
causing the pending_activation_max_lag metric to flap.

Broker side: Add grpc_max_message_size config (default 10MB) and use it
when creating the WorkerServiceClient.

Worker side: Add TASKBROKER_GRPC_MAX_MESSAGE_SIZE env var (default 10MB)
and apply it to the gRPC server options.

ref INC-2209

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@untitaker untitaker force-pushed the fix-grpc-max-message-size branch from 993fb3a to 9ec3631 Compare May 29, 2026 20:33
@untitaker untitaker merged commit e50d396 into main May 29, 2026
26 checks passed
@untitaker untitaker deleted the fix-grpc-max-message-size branch May 29, 2026 21:04
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