Skip to content

Revert "Replace work serializer with a mutex in c-ares resolver"#28324

Merged
ctiller merged 1 commit intomasterfrom
revert-27858-ares_resolver_mutex
Dec 9, 2021
Merged

Revert "Replace work serializer with a mutex in c-ares resolver"#28324
ctiller merged 1 commit intomasterfrom
revert-27858-ares_resolver_mutex

Conversation

@ctiller
Copy link
Copy Markdown
Member

@ctiller ctiller commented Dec 9, 2021

Reverts #27858

Suspect this is the problem blocking cl/415106377 from submitting

@ctiller ctiller requested a review from markdroth as a code owner December 9, 2021 16:01
@ctiller ctiller requested a review from apolcyn December 9, 2021 16:01
@ctiller ctiller added the release notes: no Indicates if PR should not be in release notes label Dec 9, 2021
@grpc-checks grpc-checks bot added the bloat/low label Dec 9, 2021
@ctiller ctiller merged commit b972b76 into master Dec 9, 2021
@ctiller ctiller deleted the revert-27858-ares_resolver_mutex branch December 9, 2021 16:26
apolcyn added a commit to apolcyn/grpc that referenced this pull request Dec 9, 2021
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Dec 9, 2021
apolcyn added a commit that referenced this pull request Dec 9, 2021
…er" #28324" (#28325)

* Revert "Revert "Replace work serializer with a mutex in c-ares resolver (#27858)" (#28324)"

This reverts commit b972b76.

* Fix synchronization of grpc_ares_resolver_address
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bloat/low imported Specifies if the PR has been imported to the internal repository lang/core perf-change/high release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants