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 <[email protected]> wrote: > > Correct. > >> On Jul 11, 2019, at 9:23 AM, Darrel Schneider <[email protected]> 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 <[email protected]> wrote: >>> >>> 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 >>>> >>> >>> >
