doesn't feel like it belongs to the factory... why not put it in the Aggregator class?