Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12749 )
Change subject: [TSManager] don't hold lock while running LA command ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/12749/1/src/kudu/master/ts_manager.cc File src/kudu/master/ts_manager.cc: http://gerrit.cloudera.org:8080/#/c/12749/1/src/kudu/master/ts_manager.cc@61 PS1, Line 61: using std::vector; > warning: using decl 'vector' is unused [misc-unused-using-decls] Done http://gerrit.cloudera.org:8080/#/c/12749/1/src/kudu/master/ts_manager.cc@143 PS1, Line 143: location.emplace(location_str); > No std::move()? Done http://gerrit.cloudera.org:8080/#/c/12749/1/src/kudu/master/ts_manager.cc@150 PS1, Line 150: if (ContainsKey(servers_by_id_, uuid)) { : descriptor = FindOrDie(servers_by_id_, uuid); > Should be able to use a Find* function to avoid the double lookup in the re Whoops, I thought the same but then missed updating this. Done. -- To view, visit http://gerrit.cloudera.org:8080/12749 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I59cd5f6ed19c162a7c9f9a6527e78cab782b4539 Gerrit-Change-Number: 12749 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Thu, 14 Mar 2019 02:05:47 +0000 Gerrit-HasComments: Yes
