gianm commented on issue #8125: Leave a single CachePopulator and choose sync/async mode automatically URL: https://github.com/apache/incubator-druid/issues/8125#issuecomment-513972679 By the way, after writing the above comment, now I'm starting to wonder if the design of the background populator makes sense. In particular, is it really a good idea to try to compute cache entries asynchronously _and in parallel_? And is it a good idea to have background cache population be a generic thing at all? What if we only had foreground population and left it up to the individual Cache impls to do background pushing to remote cache servers if they want to?
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
