You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/02/23 22:18:40 UTC

[GitHub] [pulsar] rdhabalia commented on issue #9645: Project-10: Remove ConfigurationCacheService (global-zk + local-zk) cache with Metadata-store API

rdhabalia commented on issue #9645:
URL: https://github.com/apache/pulsar/issues/9645#issuecomment-784553550


   The only issue in removing `ConfigurationCacheService` is it's used by `AuthorizationProvider::initialize` and part of the contract. We can deprecate it and eventually remove it. 
   However, `PulsarAuthorizationProvider` already uses `ConfigurationMetadataCacheService` now. So, yes, we can replace `PulsarResources` with `ConfigurationCacheService` at other places.
   
   > I wonder if there is a need to have a ConfigurationMetadataCacheService class at all, we could use directly PulsarResources
   
   yes. We can directly use `PulsarResources` where `ConfigurationCacheService` is not part of the contract.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org