Alexey Serbin has submitted this change and it was merged. ( 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 Reviewed-on: http://gerrit.cloudera.org:8080/15977 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Andrew Wong <[email protected]> --- 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(-) Approvals: Alexey Serbin: Verified Andrew Wong: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I00dd5d1f849d081e6a36190871e352e98f16f0a2 Gerrit-Change-Number: 15977 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
