Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13062 )
Change subject: [catalog_manager] simplify lifecycle of authz provider ...................................................................... [catalog_manager] simplify lifecycle of authz provider This patch simplifies CatalogManager::authz_provider_ member's lifecycle. The motivation for this change is to make the authz_provider_ member available right after construction of a CatalogManager instance: this will be used in follow-up changelists. I also did a couple of other style-related changes. Change-Id: I47a83a332dc1e55725dd46d4d9510ac4fd8d37c2 Reviewed-on: http://gerrit.cloudera.org:8080/13062 Tested-by: Alexey Serbin <aser...@cloudera.com> Reviewed-by: Adar Dembo <a...@cloudera.com> Reviewed-by: Andrew Wong <aw...@cloudera.com> Reviewed-by: Hao Hao <hao....@cloudera.com> --- M src/kudu/master/authz_provider.cc M src/kudu/master/authz_provider.h M src/kudu/master/catalog_manager.cc 3 files changed, 8 insertions(+), 11 deletions(-) Approvals: Alexey Serbin: Verified Adar Dembo: Looks good to me, approved Andrew Wong: Looks good to me, approved Hao Hao: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/13062 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I47a83a332dc1e55725dd46d4d9510ac4fd8d37c2 Gerrit-Change-Number: 13062 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120)