HBASE-7695 Remove unused member and local variables from HRegionServer (Nick Dimiduk)
git-svn-id: https://svn.apache.org/repos/asf/hbase/trunk@1440211 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
3d5de7ecbb
commit
808a3378bb
|
@ -594,17 +594,6 @@ public final class HConstants {
|
|||
*/
|
||||
public static String HBASE_CLIENT_INSTANCE_ID = "hbase.client.instance.id";
|
||||
|
||||
/**
|
||||
* The row lock timeout period in milliseconds.
|
||||
*/
|
||||
public static String HBASE_REGIONSERVER_ROWLOCK_TIMEOUT_PERIOD =
|
||||
"hbase.regionserver.rowlock.timeout.period";
|
||||
|
||||
/**
|
||||
* Default value of {@link #HBASE_REGIONSERVER_ROWLOCK_TIMEOUT_PERIOD}.
|
||||
*/
|
||||
public static int DEFAULT_HBASE_REGIONSERVER_ROWLOCK_TIMEOUT_PERIOD = 60000;
|
||||
|
||||
/**
|
||||
* The client scanner timeout period in milliseconds.
|
||||
*/
|
||||
|
|
|
@ -413,11 +413,6 @@ public class HRegionServer implements ClientProtocol,
|
|||
*/
|
||||
private MovedRegionsCleaner movedRegionsCleaner;
|
||||
|
||||
/**
|
||||
* The lease timeout period for row locks (milliseconds).
|
||||
*/
|
||||
private final int rowLockLeaseTimeoutPeriod;
|
||||
|
||||
/**
|
||||
* The lease timeout period for client scanners (milliseconds).
|
||||
*/
|
||||
|
@ -473,10 +468,6 @@ public class HRegionServer implements ClientProtocol,
|
|||
this.abortRequested = false;
|
||||
this.stopped = false;
|
||||
|
||||
this.rowLockLeaseTimeoutPeriod = conf.getInt(
|
||||
HConstants.HBASE_REGIONSERVER_ROWLOCK_TIMEOUT_PERIOD,
|
||||
HConstants.DEFAULT_HBASE_REGIONSERVER_ROWLOCK_TIMEOUT_PERIOD);
|
||||
|
||||
this.scannerLeaseTimeoutPeriod = conf.getInt(HConstants.HBASE_CLIENT_SCANNER_TIMEOUT_PERIOD,
|
||||
HConstants.DEFAULT_HBASE_CLIENT_SCANNER_TIMEOUT_PERIOD);
|
||||
|
||||
|
@ -2755,7 +2746,6 @@ public class HRegionServer implements ClientProtocol,
|
|||
if (!region.getRegionInfo().isMetaTable()) {
|
||||
cacheFlusher.reclaimMemStoreMemory();
|
||||
}
|
||||
Integer lock;
|
||||
Result r = null;
|
||||
Boolean processed = null;
|
||||
MutateType type = mutate.getMutateType();
|
||||
|
|
|
@ -166,12 +166,6 @@
|
|||
<description>Client scanner lease period in milliseconds. Default is
|
||||
60 seconds. </description>
|
||||
</property>
|
||||
<property>
|
||||
<name>hbase.regionserver.rowlock.timeout.period</name>
|
||||
<value>60000</value>
|
||||
<description>Row lock time out period in milliseconds. Default is
|
||||
60 seconds. </description>
|
||||
</property>
|
||||
<property>
|
||||
<name>hbase.regionserver.handler.count</name>
|
||||
<value>10</value>
|
||||
|
|
|
@ -27,8 +27,6 @@ import java.util.List;
|
|||
import java.util.Map;
|
||||
import java.util.concurrent.ConcurrentMap;
|
||||
|
||||
import org.apache.commons.collections.map.AbstractReferenceMap;
|
||||
import org.apache.commons.collections.map.ReferenceMap;
|
||||
import org.apache.commons.logging.Log;
|
||||
import org.apache.commons.logging.LogFactory;
|
||||
import org.apache.hadoop.conf.Configuration;
|
||||
|
@ -46,16 +44,14 @@ import org.apache.hadoop.hbase.KeyValue;
|
|||
import org.apache.hadoop.hbase.Server;
|
||||
import org.apache.hadoop.hbase.SmallTests;
|
||||
import org.apache.hadoop.hbase.client.Get;
|
||||
import org.apache.hadoop.hbase.client.Put;
|
||||
import org.apache.hadoop.hbase.client.Scan;
|
||||
import org.apache.hadoop.hbase.regionserver.HRegion;
|
||||
import org.apache.hadoop.hbase.regionserver.HStore;
|
||||
import org.apache.hadoop.hbase.regionserver.InternalScanner;
|
||||
import org.apache.hadoop.hbase.regionserver.RegionCoprocessorHost;
|
||||
import org.apache.hadoop.hbase.regionserver.RegionScanner;
|
||||
import org.apache.hadoop.hbase.regionserver.SplitTransaction;
|
||||
import org.apache.hadoop.hbase.regionserver.HStore;
|
||||
import org.apache.hadoop.hbase.regionserver.StoreFile;
|
||||
import org.apache.hadoop.hbase.regionserver.wal.WALEdit;
|
||||
import org.apache.hadoop.hbase.util.Bytes;
|
||||
import org.apache.hadoop.hbase.util.PairOfSameType;
|
||||
import org.junit.experimental.categories.Category;
|
||||
|
@ -462,8 +458,6 @@ public class TestCoprocessorInterface extends HBaseTestCase {
|
|||
TEST_UTIL.getConfiguration().setInt(
|
||||
"hbase.master.lease.thread.wakefrequency", 5 * 1000);
|
||||
TEST_UTIL.getConfiguration().setInt(HConstants.HBASE_CLIENT_SCANNER_TIMEOUT_PERIOD, 10 * 1000);
|
||||
TEST_UTIL.getConfiguration().setInt(HConstants.HBASE_REGIONSERVER_ROWLOCK_TIMEOUT_PERIOD,
|
||||
10 * 1000);
|
||||
// Increase the amount of time between client retries
|
||||
TEST_UTIL.getConfiguration().setLong("hbase.client.pause", 15 * 1000);
|
||||
// This size should make it so we always split using the addContent
|
||||
|
|
|
@ -3820,7 +3820,6 @@ public class TestHRegion extends HBaseTestCase {
|
|||
conf.setInt("hbase.master.lease.thread.wakefrequency", 5 * 1000);
|
||||
|
||||
conf.setInt(HConstants.HBASE_CLIENT_SCANNER_TIMEOUT_PERIOD, 10 * 1000);
|
||||
conf.setInt(HConstants.HBASE_REGIONSERVER_ROWLOCK_TIMEOUT_PERIOD, 10 * 1000);
|
||||
|
||||
// Increase the amount of time between client retries
|
||||
conf.setLong("hbase.client.pause", 15 * 1000);
|
||||
|
|
Loading…
Reference in New Issue