There isn’t currently a partition stat instance per bucket. Are you saying you’re making that a thing now?
> On Jul 11, 2019, at 9:24 AM, Mark Hanson <mhan...@pivotal.io> wrote: > > Correct. > >> On Jul 11, 2019, at 9:23 AM, Darrel Schneider <dschnei...@pivotal.io> wrote: >> >> 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 >>>> >>> >>> >
