It depends to be honest. This is just a plan. I would hope to roll them up, but I can see a case where you have buckets on different servers, that would seem to necessitate that.
> On Jul 11, 2019, at 9:26 AM, Jacob Barrett <[email protected]> wrote: > > 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 >>>>> >>>> >>>> >>
