I think these new methods could be made private and have all the code that deals with failedBatchRemoveMessageKeys in this class instead of in subclasses. For example addToFailedBatchRemovalMessageKeys could do the check isFailedBatchRemovalMessageKeysClearedFlag in this calls instead of having the caller do it. It is okay with me if this refactoring is done in a future PR.
[ Full content available at: https://github.com/apache/geode/pull/2375 ] This message was relayed via gitbox.apache.org for [email protected]
