Seems reasonable. I'm guessing that CachePerfImpl contains many RegionStats. Does PartitionRegionStatsImpl just contain a single RegionStats?
On Wed, Jul 10, 2019 at 4:49 PM Mark Hanson <mhan...@pivotal.io> wrote: > Hi All, > > As many of you may know our structure for our perf stats is not great. I > would like to propose we refactor the code to have the following > inheritance model, which Kirk and I came up with. > > It is my belief that fixing this will allow future features to be > implemented in a much less painful way. > > Thoughts? > > Thanks, > Mark >
