Skip to content

ASP-2043: Add serviceConfig support to GrpcChannelConfig for gRPC built-in retry#93

Open
kunalvaswani123 wants to merge 1 commit intohypertrace:mainfrom
kunalvaswani123:ASP-2043
Open

ASP-2043: Add serviceConfig support to GrpcChannelConfig for gRPC built-in retry#93
kunalvaswani123 wants to merge 1 commit intohypertrace:mainfrom
kunalvaswani123:ASP-2043

Conversation

@kunalvaswani123
Copy link
Copy Markdown

@kunalvaswani123 kunalvaswani123 commented Apr 24, 2026

Summary

  • Add optional serviceConfig: Map<String, Object> field to GrpcChannelConfig
  • When set, wire defaultServiceConfig() + enableRetry() in GrpcChannelRegistry.configureAndBuildChannel()

This allows callers to pass a gRPC service config with a retryPolicy block directly through the channel config, enabling gRPC's built-in retry without any additional interceptor dependencies.

Test plan

  • Existing grpc-client-utils tests pass
  • Verify channel builds correctly with and without serviceConfig set

🤖 Generated with Claude Code

@kunalvaswani123 kunalvaswani123 requested a review from a team as a code owner April 24, 2026 05:00
@kunalvaswani123 kunalvaswani123 changed the title ASP-2043: Add grpc-retry-utils module and service config support in GrpcChannelConfig ASP-2043: Add serviceConfig support to GrpcChannelConfig for gRPC built-in retry Apr 24, 2026
…lt-in retry

- Add optional serviceConfig field to GrpcChannelConfig
- Wire defaultServiceConfig() + enableRetry() in GrpcChannelRegistry when serviceConfig is set
- Add test verifying channels with different service configs are cached independently

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

AI-Session-Id: 75f3864c-dc2c-4e44-b4f3-b23c980f2ea6
AI-Tool: claude-code
AI-Model: unknown
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.

1 participant