Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/13062 )
Change subject: [catalog_manager] simplify lifecycle of authz provider ...................................................................... Patch Set 1: Code-Review+2 Hao and I had a related discussion about this in https://gerrit.cloudera.org/c/11797/3/src/kudu/master/catalog_manager.cc#754. I'm curious to hear how this helps follow-on work: this patch does make the authz provider not null earlier in the CatalogManager lifecycle, but the provider isn't Start()ed until the same place in Init(). What kind of functionality will find a not null but not Start()ed authz provider useful? -- 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: comment Gerrit-Change-Id: I47a83a332dc1e55725dd46d4d9510ac4fd8d37c2 Gerrit-Change-Number: 13062 Gerrit-PatchSet: 1 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) Gerrit-Comment-Date: Thu, 18 Apr 2019 18:29:43 +0000 Gerrit-HasComments: No