I don't like that this code passes in a Region and then you ask that region for a bunch of other components. Instead you should make clear what you need when creating a ReplicateTombstoneSweeper for a region. It looks like you need InternalCache, CachePerfStats, CancelCriterion, and an ExecutorService. Let the caller pass these in instead of you getting them from the region.
[ Full content available at: https://github.com/apache/geode/pull/2552 ] This message was relayed via gitbox.apache.org for [email protected]
