expose dns load balancing options through c bridge - #1251
Conversation
|
Looks like this just needs a |
Head branch was pushed to by a user without write access
|
woops, done |
|
I think this should drop the |
|
I'll be on vacation until June 8, so I created another commit with the alternate approach dropping the Feel free to use that instead (or to re-create the change, or I could also pick this up when I return) |
|
Thank you for bringing the issue to our attention and proactively providing a fix. I ended up reimplementing the change in #1267 without the |
What was changed
Added
TemporalCoreClientDnsLoadBalancingOptionsto the C bridge, so that users of the C bridge can set these options how they needWhy?
Users of the C bridge want to be able to change the dns load balancing options
Checklist
#1250
Consumed this change in the .NET SDK: temporalio/sdk-dotnet#676
Verified all tests pass (both
cargo testin this repository, anddotnet testin the dotnet-sdk`)