Revert "New SteppingRegionSplitPolicy, avoid too aggressive spread of regions for small tables."
This reverts commit 791052c50a902bda35aed30d6421fb09538119fd.
This commit is contained in:
parent
791052c50a
commit
1c07d1d591
@ -45,7 +45,7 @@ import org.apache.hadoop.hbase.classification.InterfaceAudience;
|
|||||||
public class IncreasingToUpperBoundRegionSplitPolicy extends ConstantSizeRegionSplitPolicy {
|
public class IncreasingToUpperBoundRegionSplitPolicy extends ConstantSizeRegionSplitPolicy {
|
||||||
|
|
||||||
private static final Log LOG = LogFactory.getLog(IncreasingToUpperBoundRegionSplitPolicy.class);
|
private static final Log LOG = LogFactory.getLog(IncreasingToUpperBoundRegionSplitPolicy.class);
|
||||||
protected long initialSize;
|
private long initialSize;
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
protected void configureForRegion(HRegion region) {
|
protected void configureForRegion(HRegion region) {
|
||||||
@ -116,7 +116,7 @@ public class IncreasingToUpperBoundRegionSplitPolicy extends ConstantSizeRegionS
|
|||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @return Region max size or {@code count of regions cubed * 2 * flushsize},
|
* @return Region max size or {@code count of regions cubed * flushsize},
|
||||||
* which ever is smaller; guard against there being zero regions on this server.
|
* which ever is smaller; guard against there being zero regions on this server.
|
||||||
*/
|
*/
|
||||||
protected long getSizeToCheck(final int tableRegionsCount) {
|
protected long getSizeToCheck(final int tableRegionsCount) {
|
||||||
|
@ -1,32 +0,0 @@
|
|||||||
/**
|
|
||||||
* Licensed to the Apache Software Foundation (ASF) under one
|
|
||||||
* or more contributor license agreements. See the NOTICE file
|
|
||||||
* distributed with this work for additional information
|
|
||||||
* regarding copyright ownership. The ASF licenses this file
|
|
||||||
* to you under the Apache License, Version 2.0 (the
|
|
||||||
* "License"); you may not use this file except in compliance
|
|
||||||
* with the License. You may obtain a copy of the License at
|
|
||||||
*
|
|
||||||
* http://www.apache.org/licenses/LICENSE-2.0
|
|
||||||
*
|
|
||||||
* Unless required by applicable law or agreed to in writing, software
|
|
||||||
* distributed under the License is distributed on an "AS IS" BASIS,
|
|
||||||
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
||||||
* See the License for the specific language governing permissions and
|
|
||||||
* limitations under the License.
|
|
||||||
*/
|
|
||||||
package org.apache.hadoop.hbase.regionserver;
|
|
||||||
|
|
||||||
public class SteppingSplitPolicy extends IncreasingToUpperBoundRegionSplitPolicy {
|
|
||||||
/**
|
|
||||||
* @return flushSize * 2 if there's exactly one region of the table in question
|
|
||||||
* found on this regionserver. Otherwise max file size.
|
|
||||||
* This allows a table to spread quickly across servers, while avoiding creating
|
|
||||||
* too many regions.
|
|
||||||
*/
|
|
||||||
protected long getSizeToCheck(final int tableRegionsCount) {
|
|
||||||
return tableRegionsCount == 1 ? this.initialSize : getDesiredMaxFileSize();
|
|
||||||
}
|
|
||||||
|
|
||||||
|
|
||||||
}
|
|
Loading…
x
Reference in New Issue
Block a user