mnpoonia edited a comment on issue #931: HBASE-22285 A normalizer which merges small size regions with adjacen… URL: https://github.com/apache/hbase/pull/931#issuecomment-566537250 > Another thought, the main structure for _computePlanForTable_ looks very similar to the one from _SimpleRegionNormalizer_. Would it be too difficult to refactor __SimpleRegionNormalizer_ so that both normalizer implementations could make most of code reuse? After giving it a thought i believe we should have two normalizers one which splits and second which merges and may be third SimpleRegionNormalizer which uses these two and together. And to reuse code maybe we can have a utility class for normalizer which contains only pure methods which will help other normalization implementations. @wchevreuil An example https://github.com/mnpoonia/hbase/commit/b8489d51e499b6ead224fe76b0c98a220b5a3c86
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
