PartitionRegionStatsImpl can contain one to many RegionStats
> On Jul 10, 2019, at 4:53 PM, Dan Smith <[email protected]> wrote:
>
> 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 <[email protected]
> <mailto:[email protected]>> 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
>
