Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15977
Change subject: [catalog] remove CatalogManager::CheckOnline() ...................................................................... [catalog] remove CatalogManager::CheckOnline() The implementation of CatalogManager::CheckOnline() was there to verify that the catalog is running. As it turns out, callers of the methods where CheckOnline() was invoked always (except for the code in master_path_handlers.cc) create ScopedLeaderSharedLock instance prior to a call with a follow-up verification of the catalog's status (e.g., see MasterServiceImpl). In other words, CheckOnline() was always performing a verification that has already been done. This patch removes CatalogManager::CheckOnline() along with the extra verification of the catalog's status. Change-Id: I00dd5d1f849d081e6a36190871e352e98f16f0a2 --- M src/kudu/master/catalog_manager.cc M src/kudu/master/catalog_manager.h M src/kudu/master/master_path_handlers.cc 3 files changed, 4 insertions(+), 28 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/77/15977/1 -- To view, visit http://gerrit.cloudera.org:8080/15977 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I00dd5d1f849d081e6a36190871e352e98f16f0a2 Gerrit-Change-Number: 15977 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
