Hello Alexey Serbin, Andrew Wong,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/14424
to review the following change.
Change subject: net_util: tag DNS resolution as a blocking operation
......................................................................
net_util: tag DNS resolution as a blocking operation
ThreadRestrictions tags are basically only used to prevent reactor threads
from doing IO or invoking blocking operations. For some reason we didn't
annotate DNS resolution as blocking. After doing so, I had to make a few
exceptions in existing code. They're not great, but at least they're not in
tserver reactors.
Change-Id: I90e280855bc8e6797f887da26dfa67cfaf945dcf
---
M src/kudu/client/client-internal.cc
M src/kudu/master/catalog_manager.cc
M src/kudu/util/net/net_util.cc
3 files changed, 12 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/24/14424/1
--
To view, visit http://gerrit.cloudera.org:8080/14424
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I90e280855bc8e6797f887da26dfa67cfaf945dcf
Gerrit-Change-Number: 14424
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
