EdColeman commented on PR #3588:
URL: https://github.com/apache/accumulo/pull/3588#issuecomment-1631382109
ServerConfigurationFactory maintains a cache of the configuration hierarchy.
It would be nice if we reduced caching / re-used existing caches when possible.
There are a couple of possibilities:
- Use ServerConfigurationFactory as is?
- Use caffeine in ServerConfigurationFactory and use that instead of
caching here?
- Use the caching here and eliminate ServerConfigurationFactory?
When the prop store was modified, the caching n ServerConfigurationFactory
was maintained to preserve functionality. Not necessarily because it was a
optimum configuration, but served to isolate property storage from additional
changes at that time.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
