Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11429 )
Change subject: [catalog_manager] optimization in AsyncAddReplicaTask ...................................................................... [catalog_manager] optimization in AsyncAddReplicaTask Introduced a minor optimization on AsyncAddReplicaTask::SendRequest() while populating the set of blacklisted candidates for an extra replica. With this patch, the code iterates over the UUIDs of the peers in the committed config to populate the set of the blacklisted candidates, rather than filtering all tablet servers using the IsRaftConfigMember() utility function. Change-Id: I1f46ef2e3e3f385f0abc7e02e521e03d47569413 Reviewed-on: http://gerrit.cloudera.org:8080/11429 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/master/catalog_manager.cc 1 file changed, 21 insertions(+), 12 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/11429 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I1f46ef2e3e3f385f0abc7e02e521e03d47569413 Gerrit-Change-Number: 11429 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley <[email protected]>
