Hello Alexey Serbin, Kudu Jenkins, Andrew Wong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14424
to look at the new patch set (#2).
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/master/catalog_manager.h
M src/kudu/util/net/net_util.cc
4 files changed, 13 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/24/14424/2
--
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: newpatchset
Gerrit-Change-Id: I90e280855bc8e6797f887da26dfa67cfaf945dcf
Gerrit-Change-Number: 14424
Gerrit-PatchSet: 2
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
