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: [email protected]
