Skip to content

expose DnsLoadBalancingOptions - #676

Closed
robcao wants to merge 1 commit into
temporalio:mainfrom
robcao:dns-options
Closed

robcao wants to merge 1 commit into
temporalio:mainfrom
robcao:dns-options

Conversation

@robcao

@robcao robcao commented May 6, 2026

Copy link
Copy Markdown
Contributor

What was changed

temporalio/sdk-rust#1250

DNS load balancing was introduced in temporalio/sdk-rust#1212, and it defaults to enabled for users of the C bridge.

These options should be exposed through the C bridge, so that users of the C bridge, such as .NET SDK users, can configure DNS load balancing.

Why?

.NET users want to be able to change the DNS load balancing options of the gRPC client

Checklist

  1. Closes

  2. How was this tested:

I ran the tests locally

  1. Any docs updates needed?

If the Rust side change is merged, will need to update the submodule to point to the upstream repo instead of my fork: temporalio/sdk-rust#1250

@jmaeagle99

Copy link
Copy Markdown
Contributor

Thank you for bringing the issue to our attention and proactively providing a fix.

I ended up reimplementing the change in #689 without the enabled field and aligned it with the behavior of the other SDKs. I'll close out this PR at this time since it has been superseded.

@jmaeagle99 jmaeagle99 closed this May 14, 2026
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