Ádám Bakai has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20913
Change subject: [master_runner] Destruct server before construct new one ...................................................................... [master_runner] Destruct server before construct new one ServerBase constructor runs MinidumpExceptionHandler constructor that calls RegisterMinidumpExceptionHandler(). This function increments the static atomic variable current_num_instances_. Then the ServerBase is destructed, a similar process happens and current_num_instances_ gets decremented. If current_num_instances_ is not zero before incrementing or not 1 before decrementing, then it is considered an error. This indicates that only one Server can run at any given time. But in case of multi-master config, the master server is replaced, and without the change it is possible that the second server's constructor precede first server's destructor. This change makes it sure that the destructor is executed before the second one's constructor. This patch fixes KUDU-3491 issue. Change-Id: I3c1019d092bbf9e58f4fc33753a1218bc79735d3 --- M src/kudu/master/master_runner.cc 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/13/20913/1 -- To view, visit http://gerrit.cloudera.org:8080/20913 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I3c1019d092bbf9e58f4fc33753a1218bc79735d3 Gerrit-Change-Number: 20913 Gerrit-PatchSet: 1 Gerrit-Owner: Ádám Bakai <[email protected]>
