Have you considered putting the check inside a static method of RegionSizeCalculator ?
Did you put latest patch into use in a cluster ? Thanks On Feb 6, 2014, at 4:48 AM, Lukas Nalezenec <[email protected]> wrote: > Right, the instantiation could be skipped but I would prefer keeping it as it > is. > This was my intentional design decision - all functionality should be > encapsulated inside class. > Lukas > > On 3.2.2014 19:37, Ted Yu wrote: >> The new config is checked inside RegionSizeCalculator ctor. >> Instantiation of RegionSizeCalculator can be skipped if the config says >> disabled, right ? >
