I can see why you might need a collection of all the ReplicateTimbstoneSweeper instances in this class. But I don't think you need to have a reference to every Region instance. If a region has its own ReplicateTombstoneSweeper couldn't the region keep a reference to it so that it does not need to constantly look it up? In that case this could just be a HashSet of ReplicateTombstoneSweeper instances instead of a map.
[ Full content available at: https://github.com/apache/geode/pull/2552 ] This message was relayed via gitbox.apache.org for [email protected]
