If possible redesign so these could be "final" again. This would require you to 
know the tombstone timeout when you create a per region sweeper.

If this is not possible then rename these to not be all caps which is reserved 
for final constants. Since they were being set in the constructor they are not 
"constants" anyway (even though they were final) so I recommend you change them 
to be "expiryTime" and "purgeInterval" anyway. 

[ Full content available at: https://github.com/apache/geode/pull/2552 ]
This message was relayed via gitbox.apache.org for 
[email protected]

Reply via email to