Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12749
Change subject: [TSManager] don't hold lock while running LA command ...................................................................... [TSManager] don't hold lock while running LA command This patch modifies the way how the location assignment is run during the course of tablet server registration with master. Prior to this patch, TSManager's lock on the registry of all tablet servers was held while running the location assignment command. That might lead to stacking many TS registration heartbeats in flight while master's service threads are waiting to acquire the TSManager's lock. This patch introduces changes to run the location assignment command without holding the TSManager's lock, which makes the concurrent registration of tablet servers more robust and make the processing of tablet servers' heartbeats more robust overall. Change-Id: I59cd5f6ed19c162a7c9f9a6527e78cab782b4539 --- M src/kudu/master/location_cache-test.cc M src/kudu/master/master.cc M src/kudu/master/ts_descriptor-test.cc M src/kudu/master/ts_descriptor.cc M src/kudu/master/ts_descriptor.h M src/kudu/master/ts_manager.cc 6 files changed, 123 insertions(+), 141 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/12749/1 -- 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: newchange Gerrit-Change-Id: I59cd5f6ed19c162a7c9f9a6527e78cab782b4539 Gerrit-Change-Number: 12749 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
