Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14424 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/14424 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Kudu Jenkins --- 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(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- 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: merged Gerrit-Change-Id: I90e280855bc8e6797f887da26dfa67cfaf945dcf Gerrit-Change-Number: 14424 Gerrit-PatchSet: 3 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
