Why was this moved from AbstractAggregatorImpl?  Seems like it could have remained there, just like newCacheManager()