I vote for #3, and it should be applied to all active code lines.
On Fri, Jun 19, 2020 at 3:35 AM Wellington Chevreuil < wellington.chevre...@gmail.com> wrote: > While going through the changes proposed on HBASE-24530, we > observed IncreasingToUpperBoundRegionSplitPolicy > compares hbase.hregion.max.filesize against individual stores within a > region when deciding whether to split a region or not. For tables having > multiple families, this can lead to regions much larger than what's > defined by hbase.hregion.max.filesize. > > Current proposal on HBASE-24530 is to add an extra policy that actually > compares the overall region size (combining all region stores sizes) > against hbase.hregion.max.filesize, but I wonder if it really makes sense > to keep a policy with current IncreasingToUpperBoundRegionSplitPolicy > behaviour. Would like to hear folks opinions if we should take any of the > below actions? > 1) Leave IncreasingToUpperBoundRegionSplitPolicy as it is and just add the > new policy proposed on HBASE-24530; > 2) Make IncreasingToUpperBoundRegionSplitPolicy deprecated and remove it > from master branch; > 3) Change IncreasingToUpperBoundRegionSplitPolicy to actually implement the > logic of the new policy proposed on HBASE-24530; > > My view is that the current IncreasingToUpperBoundRegionSplitPolicy > behaviour is a bug, and I vote for #3. > -- Best regards, Andrew Words like orphans lost among the crosstalk, meaning torn from truth's decrepit hands - A23, Crosstalk