Better readability of - hbase.regionserver.lease.period 
--------------------------------------------------------

                 Key: HBASE-2193
                 URL: https://issues.apache.org/jira/browse/HBASE-2193
             Project: Hadoop HBase
          Issue Type: Improvement
            Reporter: Kay Kay


Property - "hbase.regionserver.lease.period" hardcode would be ideally moved to 
HConstants. 

Also - default value seems to be listed all over the place, (thankfully 
consistent though). That would be nice to be in HConstants too .. 

$ find . -name \*.java | xargs grep -nr hbase.regionserver.lease.period
./src/contrib/mdc_replication/src/java/org/apache/hadoop/hbase/replication/ReplicationConnectionManager.java:59:
    this.rpcTimeout = conf.getLong("hbase.regionserver.lease.period", 60000);
./src/java/org/apache/hadoop/hbase/client/HConnectionManager.java:298:      
this.rpcTimeout = conf.getLong("hbase.regionserver.lease.period", 60000);
./src/java/org/apache/hadoop/hbase/client/HTable.java:121:      
conf.getInt("hbase.regionserver.lease.period", 60 * 1000);
./src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java:271:    
this.rpcTimeout = conf.getLong("hbase.regionserver.lease.period", 60000);
./src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java:335:        
conf.getInt("hbase.regionserver.lease.period", 60 * 1000),


./src/test/org/apache/hadoop/hbase/regionserver/TestHRegion.java:1890:    
conf.setInt("hbase.regionserver.lease.period", 10 * 1000);


More of a nit, but provides better readability across. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to