Details
-
Sub-task
-
Status: Resolved
-
Major
-
Resolution: Incomplete
-
None
-
None
-
None
Description
This is a fair amount of duplication and inconsistency in the meta cache handling of RetryingCallable implementations:
- meta cache is often cleared in prepare() when reload=true, in addition to being cleared in throwable()
- each RetryingCallable implementation does this slightly differently, leading to inconsistencies and potential bugs
- RegionServerCallable and RegionAdminServiceCallable duplicate a lot of code, but with small, seemingly unnecessary inconsistencies. We should clean these up into a common base with subclasses doing only the necessary differentiation.
The main goal here is to establish some common handling, to the extent possible, for the meta cache interactions by the different implementations.