Why would a PartitionedRegionStatsImpl contain more than one RegionStats? Are these representing the local buckets?
On Wed, Jul 10, 2019 at 4:57 PM Mark Hanson <mhan...@pivotal.io> wrote: > PartitionRegionStatsImpl can contain one to many RegionStats > > > On Jul 10, 2019, at 4:53 PM, Dan Smith <dsm...@pivotal.io> 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 <mhan...@pivotal.io <mailto: > 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 > > > >
