http://git-wip-us.apache.org/repos/asf/hbase-site/blob/1837997e/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/AnnotationReadingPriorityFunction.html
----------------------------------------------------------------------
diff --git 
a/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/AnnotationReadingPriorityFunction.html
 
b/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/AnnotationReadingPriorityFunction.html
index 7e411c6..39d8ea2 100644
--- 
a/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/AnnotationReadingPriorityFunction.html
+++ 
b/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/AnnotationReadingPriorityFunction.html
@@ -41,255 +41,253 @@
 <span class="sourceLineNo">033</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.AdminProtos.FlushRegionRequest;<a
 name="line.33"></a>
 <span class="sourceLineNo">034</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.AdminProtos.GetRegionInfoRequest;<a
 name="line.34"></a>
 <span class="sourceLineNo">035</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.AdminProtos.GetStoreFileRequest;<a
 name="line.35"></a>
-<span class="sourceLineNo">036</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.AdminProtos.SplitRegionRequest;<a
 name="line.36"></a>
-<span class="sourceLineNo">037</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.ClientProtos.GetRequest;<a 
name="line.37"></a>
-<span class="sourceLineNo">038</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.ClientProtos.MutateRequest;<a 
name="line.38"></a>
-<span class="sourceLineNo">039</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.ClientProtos.ScanRequest;<a 
name="line.39"></a>
-<span class="sourceLineNo">040</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.HBaseProtos.RegionSpecifier;<a
 name="line.40"></a>
-<span class="sourceLineNo">041</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.RPCProtos.RequestHeader;<a 
name="line.41"></a>
-<span class="sourceLineNo">042</span><a name="line.42"></a>
-<span class="sourceLineNo">043</span>import 
org.apache.hadoop.hbase.shaded.com.google.common.annotations.VisibleForTesting;<a
 name="line.43"></a>
-<span class="sourceLineNo">044</span>import 
org.apache.hadoop.hbase.shaded.com.google.protobuf.Message;<a 
name="line.44"></a>
-<span class="sourceLineNo">045</span>import 
org.apache.hadoop.hbase.shaded.com.google.protobuf.TextFormat;<a 
name="line.45"></a>
-<span class="sourceLineNo">046</span>import 
org.apache.hadoop.hbase.security.User;<a name="line.46"></a>
-<span class="sourceLineNo">047</span><a name="line.47"></a>
-<span class="sourceLineNo">048</span>/**<a name="line.48"></a>
-<span class="sourceLineNo">049</span> * Reads special method annotations and 
table names to figure a priority for use by QoS facility in<a 
name="line.49"></a>
-<span class="sourceLineNo">050</span> * ipc; e.g: rpcs to hbase:meta get 
priority.<a name="line.50"></a>
-<span class="sourceLineNo">051</span> */<a name="line.51"></a>
-<span class="sourceLineNo">052</span>// TODO: Remove.  This is doing way too 
much work just to figure a priority.  Do as Elliott<a name="line.52"></a>
-<span class="sourceLineNo">053</span>// suggests and just have the client 
specify a priority.<a name="line.53"></a>
-<span class="sourceLineNo">054</span><a name="line.54"></a>
-<span class="sourceLineNo">055</span>//The logic for figuring out high 
priority RPCs is as follows:<a name="line.55"></a>
-<span class="sourceLineNo">056</span>//1. if the method is annotated with a 
QosPriority of QOS_HIGH,<a name="line.56"></a>
-<span class="sourceLineNo">057</span>//   that is honored<a name="line.57"></a>
-<span class="sourceLineNo">058</span>//2. parse out the protobuf message and 
see if the request is for meta<a name="line.58"></a>
-<span class="sourceLineNo">059</span>//   region, and if so, treat it as a 
high priority RPC<a name="line.59"></a>
-<span class="sourceLineNo">060</span>//Some optimizations for (2) are done 
here -<a name="line.60"></a>
-<span class="sourceLineNo">061</span>//Clients send the argument classname as 
part of making the RPC. The server<a name="line.61"></a>
-<span class="sourceLineNo">062</span>//decides whether to deserialize the 
proto argument message based on the<a name="line.62"></a>
-<span class="sourceLineNo">063</span>//pre-established set of argument classes 
(knownArgumentClasses below).<a name="line.63"></a>
-<span class="sourceLineNo">064</span>//This prevents the server from having to 
deserialize all proto argument<a name="line.64"></a>
-<span class="sourceLineNo">065</span>//messages prematurely.<a 
name="line.65"></a>
-<span class="sourceLineNo">066</span>//All the argument classes declare a 
'getRegion' method that returns a<a name="line.66"></a>
-<span class="sourceLineNo">067</span>//RegionSpecifier object. Methods can be 
invoked on the returned object<a name="line.67"></a>
-<span class="sourceLineNo">068</span>//to figure out whether it is a meta 
region or not.<a name="line.68"></a>
-<span class="sourceLineNo">069</span>@InterfaceAudience.Private<a 
name="line.69"></a>
-<span class="sourceLineNo">070</span>public class 
AnnotationReadingPriorityFunction implements PriorityFunction {<a 
name="line.70"></a>
-<span class="sourceLineNo">071</span>  private static final Log LOG =<a 
name="line.71"></a>
-<span class="sourceLineNo">072</span>    
LogFactory.getLog(AnnotationReadingPriorityFunction.class.getName());<a 
name="line.72"></a>
-<span class="sourceLineNo">073</span><a name="line.73"></a>
-<span class="sourceLineNo">074</span>  /** Used to control the scan delay, 
currently sqrt(numNextCall * weight) */<a name="line.74"></a>
-<span class="sourceLineNo">075</span>  public static final String 
SCAN_VTIME_WEIGHT_CONF_KEY = "hbase.ipc.server.scan.vtime.weight";<a 
name="line.75"></a>
-<span class="sourceLineNo">076</span><a name="line.76"></a>
-<span class="sourceLineNo">077</span>  protected final Map&lt;String, 
Integer&gt; annotatedQos;<a name="line.77"></a>
-<span class="sourceLineNo">078</span>  //We need to mock the regionserver 
instance for some unit tests (set via<a name="line.78"></a>
-<span class="sourceLineNo">079</span>  //setRegionServer method.<a 
name="line.79"></a>
-<span class="sourceLineNo">080</span>  private RSRpcServices rpcServices;<a 
name="line.80"></a>
-<span class="sourceLineNo">081</span>  @SuppressWarnings("unchecked")<a 
name="line.81"></a>
-<span class="sourceLineNo">082</span>  private final Class&lt;? extends 
Message&gt;[] knownArgumentClasses = new Class[]{<a name="line.82"></a>
-<span class="sourceLineNo">083</span>      GetRegionInfoRequest.class,<a 
name="line.83"></a>
-<span class="sourceLineNo">084</span>      GetStoreFileRequest.class,<a 
name="line.84"></a>
-<span class="sourceLineNo">085</span>      CloseRegionRequest.class,<a 
name="line.85"></a>
-<span class="sourceLineNo">086</span>      FlushRegionRequest.class,<a 
name="line.86"></a>
-<span class="sourceLineNo">087</span>      SplitRegionRequest.class,<a 
name="line.87"></a>
-<span class="sourceLineNo">088</span>      CompactRegionRequest.class,<a 
name="line.88"></a>
-<span class="sourceLineNo">089</span>      GetRequest.class,<a 
name="line.89"></a>
-<span class="sourceLineNo">090</span>      MutateRequest.class,<a 
name="line.90"></a>
-<span class="sourceLineNo">091</span>      ScanRequest.class<a 
name="line.91"></a>
-<span class="sourceLineNo">092</span>  };<a name="line.92"></a>
-<span class="sourceLineNo">093</span><a name="line.93"></a>
-<span class="sourceLineNo">094</span>  // Some caches for helping 
performance<a name="line.94"></a>
-<span class="sourceLineNo">095</span>  private final Map&lt;String, Class&lt;? 
extends Message&gt;&gt; argumentToClassMap = new HashMap&lt;&gt;();<a 
name="line.95"></a>
-<span class="sourceLineNo">096</span>  private final Map&lt;String, 
Map&lt;Class&lt;? extends Message&gt;, Method&gt;&gt; methodMap = new 
HashMap&lt;&gt;();<a name="line.96"></a>
+<span class="sourceLineNo">036</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.ClientProtos.GetRequest;<a 
name="line.36"></a>
+<span class="sourceLineNo">037</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.ClientProtos.MutateRequest;<a 
name="line.37"></a>
+<span class="sourceLineNo">038</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.ClientProtos.ScanRequest;<a 
name="line.38"></a>
+<span class="sourceLineNo">039</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.HBaseProtos.RegionSpecifier;<a
 name="line.39"></a>
+<span class="sourceLineNo">040</span>import 
org.apache.hadoop.hbase.shaded.protobuf.generated.RPCProtos.RequestHeader;<a 
name="line.40"></a>
+<span class="sourceLineNo">041</span><a name="line.41"></a>
+<span class="sourceLineNo">042</span>import 
org.apache.hadoop.hbase.shaded.com.google.common.annotations.VisibleForTesting;<a
 name="line.42"></a>
+<span class="sourceLineNo">043</span>import 
org.apache.hadoop.hbase.shaded.com.google.protobuf.Message;<a 
name="line.43"></a>
+<span class="sourceLineNo">044</span>import 
org.apache.hadoop.hbase.shaded.com.google.protobuf.TextFormat;<a 
name="line.44"></a>
+<span class="sourceLineNo">045</span>import 
org.apache.hadoop.hbase.security.User;<a name="line.45"></a>
+<span class="sourceLineNo">046</span><a name="line.46"></a>
+<span class="sourceLineNo">047</span>/**<a name="line.47"></a>
+<span class="sourceLineNo">048</span> * Reads special method annotations and 
table names to figure a priority for use by QoS facility in<a 
name="line.48"></a>
+<span class="sourceLineNo">049</span> * ipc; e.g: rpcs to hbase:meta get 
priority.<a name="line.49"></a>
+<span class="sourceLineNo">050</span> */<a name="line.50"></a>
+<span class="sourceLineNo">051</span>// TODO: Remove.  This is doing way too 
much work just to figure a priority.  Do as Elliott<a name="line.51"></a>
+<span class="sourceLineNo">052</span>// suggests and just have the client 
specify a priority.<a name="line.52"></a>
+<span class="sourceLineNo">053</span><a name="line.53"></a>
+<span class="sourceLineNo">054</span>//The logic for figuring out high 
priority RPCs is as follows:<a name="line.54"></a>
+<span class="sourceLineNo">055</span>//1. if the method is annotated with a 
QosPriority of QOS_HIGH,<a name="line.55"></a>
+<span class="sourceLineNo">056</span>//   that is honored<a name="line.56"></a>
+<span class="sourceLineNo">057</span>//2. parse out the protobuf message and 
see if the request is for meta<a name="line.57"></a>
+<span class="sourceLineNo">058</span>//   region, and if so, treat it as a 
high priority RPC<a name="line.58"></a>
+<span class="sourceLineNo">059</span>//Some optimizations for (2) are done 
here -<a name="line.59"></a>
+<span class="sourceLineNo">060</span>//Clients send the argument classname as 
part of making the RPC. The server<a name="line.60"></a>
+<span class="sourceLineNo">061</span>//decides whether to deserialize the 
proto argument message based on the<a name="line.61"></a>
+<span class="sourceLineNo">062</span>//pre-established set of argument classes 
(knownArgumentClasses below).<a name="line.62"></a>
+<span class="sourceLineNo">063</span>//This prevents the server from having to 
deserialize all proto argument<a name="line.63"></a>
+<span class="sourceLineNo">064</span>//messages prematurely.<a 
name="line.64"></a>
+<span class="sourceLineNo">065</span>//All the argument classes declare a 
'getRegion' method that returns a<a name="line.65"></a>
+<span class="sourceLineNo">066</span>//RegionSpecifier object. Methods can be 
invoked on the returned object<a name="line.66"></a>
+<span class="sourceLineNo">067</span>//to figure out whether it is a meta 
region or not.<a name="line.67"></a>
+<span class="sourceLineNo">068</span>@InterfaceAudience.Private<a 
name="line.68"></a>
+<span class="sourceLineNo">069</span>public class 
AnnotationReadingPriorityFunction implements PriorityFunction {<a 
name="line.69"></a>
+<span class="sourceLineNo">070</span>  private static final Log LOG =<a 
name="line.70"></a>
+<span class="sourceLineNo">071</span>    
LogFactory.getLog(AnnotationReadingPriorityFunction.class.getName());<a 
name="line.71"></a>
+<span class="sourceLineNo">072</span><a name="line.72"></a>
+<span class="sourceLineNo">073</span>  /** Used to control the scan delay, 
currently sqrt(numNextCall * weight) */<a name="line.73"></a>
+<span class="sourceLineNo">074</span>  public static final String 
SCAN_VTIME_WEIGHT_CONF_KEY = "hbase.ipc.server.scan.vtime.weight";<a 
name="line.74"></a>
+<span class="sourceLineNo">075</span><a name="line.75"></a>
+<span class="sourceLineNo">076</span>  protected final Map&lt;String, 
Integer&gt; annotatedQos;<a name="line.76"></a>
+<span class="sourceLineNo">077</span>  //We need to mock the regionserver 
instance for some unit tests (set via<a name="line.77"></a>
+<span class="sourceLineNo">078</span>  //setRegionServer method.<a 
name="line.78"></a>
+<span class="sourceLineNo">079</span>  private RSRpcServices rpcServices;<a 
name="line.79"></a>
+<span class="sourceLineNo">080</span>  @SuppressWarnings("unchecked")<a 
name="line.80"></a>
+<span class="sourceLineNo">081</span>  private final Class&lt;? extends 
Message&gt;[] knownArgumentClasses = new Class[]{<a name="line.81"></a>
+<span class="sourceLineNo">082</span>      GetRegionInfoRequest.class,<a 
name="line.82"></a>
+<span class="sourceLineNo">083</span>      GetStoreFileRequest.class,<a 
name="line.83"></a>
+<span class="sourceLineNo">084</span>      CloseRegionRequest.class,<a 
name="line.84"></a>
+<span class="sourceLineNo">085</span>      FlushRegionRequest.class,<a 
name="line.85"></a>
+<span class="sourceLineNo">086</span>      CompactRegionRequest.class,<a 
name="line.86"></a>
+<span class="sourceLineNo">087</span>      GetRequest.class,<a 
name="line.87"></a>
+<span class="sourceLineNo">088</span>      MutateRequest.class,<a 
name="line.88"></a>
+<span class="sourceLineNo">089</span>      ScanRequest.class<a 
name="line.89"></a>
+<span class="sourceLineNo">090</span>  };<a name="line.90"></a>
+<span class="sourceLineNo">091</span><a name="line.91"></a>
+<span class="sourceLineNo">092</span>  // Some caches for helping 
performance<a name="line.92"></a>
+<span class="sourceLineNo">093</span>  private final Map&lt;String, Class&lt;? 
extends Message&gt;&gt; argumentToClassMap = new HashMap&lt;&gt;();<a 
name="line.93"></a>
+<span class="sourceLineNo">094</span>  private final Map&lt;String, 
Map&lt;Class&lt;? extends Message&gt;, Method&gt;&gt; methodMap = new 
HashMap&lt;&gt;();<a name="line.94"></a>
+<span class="sourceLineNo">095</span><a name="line.95"></a>
+<span class="sourceLineNo">096</span>  private final float 
scanVirtualTimeWeight;<a name="line.96"></a>
 <span class="sourceLineNo">097</span><a name="line.97"></a>
-<span class="sourceLineNo">098</span>  private final float 
scanVirtualTimeWeight;<a name="line.98"></a>
-<span class="sourceLineNo">099</span><a name="line.99"></a>
-<span class="sourceLineNo">100</span>  /**<a name="line.100"></a>
-<span class="sourceLineNo">101</span>   * Calls {@link 
#AnnotationReadingPriorityFunction(RSRpcServices, Class)} using the result of<a 
name="line.101"></a>
-<span class="sourceLineNo">102</span>   * {@code rpcServices#getClass()}<a 
name="line.102"></a>
-<span class="sourceLineNo">103</span>   *<a name="line.103"></a>
-<span class="sourceLineNo">104</span>   * @param rpcServices<a 
name="line.104"></a>
-<span class="sourceLineNo">105</span>   *          The RPC server 
implementation<a name="line.105"></a>
-<span class="sourceLineNo">106</span>   */<a name="line.106"></a>
-<span class="sourceLineNo">107</span>  public 
AnnotationReadingPriorityFunction(final RSRpcServices rpcServices) {<a 
name="line.107"></a>
-<span class="sourceLineNo">108</span>    this(rpcServices, 
rpcServices.getClass());<a name="line.108"></a>
-<span class="sourceLineNo">109</span>  }<a name="line.109"></a>
-<span class="sourceLineNo">110</span><a name="line.110"></a>
-<span class="sourceLineNo">111</span>  /**<a name="line.111"></a>
-<span class="sourceLineNo">112</span>   * Constructs the priority function 
given the RPC server implementation and the annotations on the<a 
name="line.112"></a>
-<span class="sourceLineNo">113</span>   * methods in the provided {@code 
clz}.<a name="line.113"></a>
-<span class="sourceLineNo">114</span>   *<a name="line.114"></a>
-<span class="sourceLineNo">115</span>   * @param rpcServices<a 
name="line.115"></a>
-<span class="sourceLineNo">116</span>   *          The RPC server 
implementation<a name="line.116"></a>
-<span class="sourceLineNo">117</span>   * @param clz<a name="line.117"></a>
-<span class="sourceLineNo">118</span>   *          The concrete RPC server 
implementation's class<a name="line.118"></a>
-<span class="sourceLineNo">119</span>   */<a name="line.119"></a>
-<span class="sourceLineNo">120</span>  public 
AnnotationReadingPriorityFunction(final RSRpcServices rpcServices,<a 
name="line.120"></a>
-<span class="sourceLineNo">121</span>      Class&lt;? extends 
RSRpcServices&gt; clz) {<a name="line.121"></a>
-<span class="sourceLineNo">122</span>    Map&lt;String,Integer&gt; qosMap = 
new HashMap&lt;&gt;();<a name="line.122"></a>
-<span class="sourceLineNo">123</span>    for (Method m : clz.getMethods()) {<a 
name="line.123"></a>
-<span class="sourceLineNo">124</span>      QosPriority p = 
m.getAnnotation(QosPriority.class);<a name="line.124"></a>
-<span class="sourceLineNo">125</span>      if (p != null) {<a 
name="line.125"></a>
-<span class="sourceLineNo">126</span>        // Since we protobuf'd, and then 
subsequently, when we went with pb style, method names<a name="line.126"></a>
-<span class="sourceLineNo">127</span>        // are capitalized.  This meant 
that this brittle compare of method names gotten by<a name="line.127"></a>
-<span class="sourceLineNo">128</span>        // reflection no longer matched 
the method names coming in over pb.  TODO: Get rid of this<a 
name="line.128"></a>
-<span class="sourceLineNo">129</span>        // check.  For now, workaround is 
to capitalize the names we got from reflection so they<a name="line.129"></a>
-<span class="sourceLineNo">130</span>        // have chance of matching the pb 
ones.<a name="line.130"></a>
-<span class="sourceLineNo">131</span>        String capitalizedMethodName = 
capitalize(m.getName());<a name="line.131"></a>
-<span class="sourceLineNo">132</span>        qosMap.put(capitalizedMethodName, 
p.priority());<a name="line.132"></a>
-<span class="sourceLineNo">133</span>      }<a name="line.133"></a>
-<span class="sourceLineNo">134</span>    }<a name="line.134"></a>
-<span class="sourceLineNo">135</span>    this.rpcServices = rpcServices;<a 
name="line.135"></a>
-<span class="sourceLineNo">136</span>    this.annotatedQos = qosMap;<a 
name="line.136"></a>
-<span class="sourceLineNo">137</span>    if (methodMap.get("getRegion") == 
null) {<a name="line.137"></a>
-<span class="sourceLineNo">138</span>      methodMap.put("hasRegion", new 
HashMap&lt;&gt;());<a name="line.138"></a>
-<span class="sourceLineNo">139</span>      methodMap.put("getRegion", new 
HashMap&lt;&gt;());<a name="line.139"></a>
-<span class="sourceLineNo">140</span>    }<a name="line.140"></a>
-<span class="sourceLineNo">141</span>    for (Class&lt;? extends Message&gt; 
cls : knownArgumentClasses) {<a name="line.141"></a>
-<span class="sourceLineNo">142</span>      
argumentToClassMap.put(cls.getName(), cls);<a name="line.142"></a>
-<span class="sourceLineNo">143</span>      try {<a name="line.143"></a>
-<span class="sourceLineNo">144</span>        
methodMap.get("hasRegion").put(cls, cls.getDeclaredMethod("hasRegion"));<a 
name="line.144"></a>
-<span class="sourceLineNo">145</span>        
methodMap.get("getRegion").put(cls, cls.getDeclaredMethod("getRegion"));<a 
name="line.145"></a>
-<span class="sourceLineNo">146</span>      } catch (Exception e) {<a 
name="line.146"></a>
-<span class="sourceLineNo">147</span>        throw new RuntimeException(e);<a 
name="line.147"></a>
-<span class="sourceLineNo">148</span>      }<a name="line.148"></a>
-<span class="sourceLineNo">149</span>    }<a name="line.149"></a>
-<span class="sourceLineNo">150</span><a name="line.150"></a>
-<span class="sourceLineNo">151</span>    Configuration conf = 
rpcServices.getConfiguration();<a name="line.151"></a>
-<span class="sourceLineNo">152</span>    scanVirtualTimeWeight = 
conf.getFloat(SCAN_VTIME_WEIGHT_CONF_KEY, 1.0f);<a name="line.152"></a>
-<span class="sourceLineNo">153</span>  }<a name="line.153"></a>
-<span class="sourceLineNo">154</span><a name="line.154"></a>
-<span class="sourceLineNo">155</span>  private String capitalize(final String 
s) {<a name="line.155"></a>
-<span class="sourceLineNo">156</span>    StringBuilder strBuilder = new 
StringBuilder(s);<a name="line.156"></a>
-<span class="sourceLineNo">157</span>    strBuilder.setCharAt(0, 
Character.toUpperCase(strBuilder.charAt(0)));<a name="line.157"></a>
-<span class="sourceLineNo">158</span>    return strBuilder.toString();<a 
name="line.158"></a>
-<span class="sourceLineNo">159</span>  }<a name="line.159"></a>
-<span class="sourceLineNo">160</span><a name="line.160"></a>
-<span class="sourceLineNo">161</span>  /**<a name="line.161"></a>
-<span class="sourceLineNo">162</span>   * Returns a 'priority' based on the 
request type.<a name="line.162"></a>
-<span class="sourceLineNo">163</span>   *<a name="line.163"></a>
-<span class="sourceLineNo">164</span>   * Currently the returned priority is 
used for queue selection.<a name="line.164"></a>
-<span class="sourceLineNo">165</span>   * See the SimpleRpcScheduler as 
example. It maintains a queue per 'priory type'<a name="line.165"></a>
-<span class="sourceLineNo">166</span>   * HIGH_QOS (meta requests), 
REPLICATION_QOS (replication requests),<a name="line.166"></a>
-<span class="sourceLineNo">167</span>   * NORMAL_QOS (user requests).<a 
name="line.167"></a>
-<span class="sourceLineNo">168</span>   */<a name="line.168"></a>
-<span class="sourceLineNo">169</span>  @Override<a name="line.169"></a>
-<span class="sourceLineNo">170</span>  public int getPriority(RequestHeader 
header, Message param, User user) {<a name="line.170"></a>
-<span class="sourceLineNo">171</span>    int priorityByAnnotation = 
getAnnotatedPriority(header);<a name="line.171"></a>
-<span class="sourceLineNo">172</span><a name="line.172"></a>
-<span class="sourceLineNo">173</span>    if (priorityByAnnotation &gt;= 0) {<a 
name="line.173"></a>
-<span class="sourceLineNo">174</span>      return priorityByAnnotation;<a 
name="line.174"></a>
-<span class="sourceLineNo">175</span>    }<a name="line.175"></a>
-<span class="sourceLineNo">176</span>    return getBasePriority(header, 
param);<a name="line.176"></a>
-<span class="sourceLineNo">177</span>  }<a name="line.177"></a>
-<span class="sourceLineNo">178</span><a name="line.178"></a>
-<span class="sourceLineNo">179</span>  /**<a name="line.179"></a>
-<span class="sourceLineNo">180</span>   * See if the method has an 
annotation.<a name="line.180"></a>
-<span class="sourceLineNo">181</span>   * @param header<a name="line.181"></a>
-<span class="sourceLineNo">182</span>   * @return Return the priority from the 
annotation. If there isn't<a name="line.182"></a>
-<span class="sourceLineNo">183</span>   * an annotation, this returns 
something below zero.<a name="line.183"></a>
-<span class="sourceLineNo">184</span>   */<a name="line.184"></a>
-<span class="sourceLineNo">185</span>  protected int 
getAnnotatedPriority(RequestHeader header) {<a name="line.185"></a>
-<span class="sourceLineNo">186</span>    String methodName = 
header.getMethodName();<a name="line.186"></a>
-<span class="sourceLineNo">187</span>    Integer priorityByAnnotation = 
annotatedQos.get(methodName);<a name="line.187"></a>
-<span class="sourceLineNo">188</span>    if (priorityByAnnotation != null) {<a 
name="line.188"></a>
-<span class="sourceLineNo">189</span>      return priorityByAnnotation;<a 
name="line.189"></a>
-<span class="sourceLineNo">190</span>    }<a name="line.190"></a>
-<span class="sourceLineNo">191</span>    return -1;<a name="line.191"></a>
-<span class="sourceLineNo">192</span>  }<a name="line.192"></a>
-<span class="sourceLineNo">193</span><a name="line.193"></a>
-<span class="sourceLineNo">194</span>  /**<a name="line.194"></a>
-<span class="sourceLineNo">195</span>   * Get the priority for a given request 
from the header and the param<a name="line.195"></a>
-<span class="sourceLineNo">196</span>   * This doesn't consider which user is 
sending the request at all.<a name="line.196"></a>
-<span class="sourceLineNo">197</span>   * This doesn't consider annotations<a 
name="line.197"></a>
-<span class="sourceLineNo">198</span>   */<a name="line.198"></a>
-<span class="sourceLineNo">199</span>  protected int 
getBasePriority(RequestHeader header, Message param) {<a name="line.199"></a>
-<span class="sourceLineNo">200</span>    if (param == null) {<a 
name="line.200"></a>
-<span class="sourceLineNo">201</span>      return HConstants.NORMAL_QOS;<a 
name="line.201"></a>
-<span class="sourceLineNo">202</span>    }<a name="line.202"></a>
-<span class="sourceLineNo">203</span><a name="line.203"></a>
-<span class="sourceLineNo">204</span>    // Trust the client-set priorities if 
set<a name="line.204"></a>
-<span class="sourceLineNo">205</span>    if (header.hasPriority()) {<a 
name="line.205"></a>
-<span class="sourceLineNo">206</span>      return header.getPriority();<a 
name="line.206"></a>
-<span class="sourceLineNo">207</span>    }<a name="line.207"></a>
-<span class="sourceLineNo">208</span><a name="line.208"></a>
-<span class="sourceLineNo">209</span>    String cls = 
param.getClass().getName();<a name="line.209"></a>
-<span class="sourceLineNo">210</span>    Class&lt;? extends Message&gt; 
rpcArgClass = argumentToClassMap.get(cls);<a name="line.210"></a>
-<span class="sourceLineNo">211</span>    RegionSpecifier regionSpecifier = 
null;<a name="line.211"></a>
-<span class="sourceLineNo">212</span>    //check whether the request has 
reference to meta region or now.<a name="line.212"></a>
-<span class="sourceLineNo">213</span>    try {<a name="line.213"></a>
-<span class="sourceLineNo">214</span>      // Check if the param has a region 
specifier; the pb methods are hasRegion and getRegion if<a name="line.214"></a>
-<span class="sourceLineNo">215</span>      // hasRegion returns true.  Not all 
listed methods have region specifier each time.  For<a name="line.215"></a>
-<span class="sourceLineNo">216</span>      // example, the ScanRequest has it 
on setup but thereafter relies on the scannerid rather than<a 
name="line.216"></a>
-<span class="sourceLineNo">217</span>      // send the region over every 
time.<a name="line.217"></a>
-<span class="sourceLineNo">218</span>      Method hasRegion = 
methodMap.get("hasRegion").get(rpcArgClass);<a name="line.218"></a>
-<span class="sourceLineNo">219</span>      if (hasRegion != null &amp;&amp; 
(Boolean)hasRegion.invoke(param, (Object[])null)) {<a name="line.219"></a>
-<span class="sourceLineNo">220</span>        Method getRegion = 
methodMap.get("getRegion").get(rpcArgClass);<a name="line.220"></a>
-<span class="sourceLineNo">221</span>        regionSpecifier = 
(RegionSpecifier)getRegion.invoke(param, (Object[])null);<a name="line.221"></a>
-<span class="sourceLineNo">222</span>        Region region = 
rpcServices.getRegion(regionSpecifier);<a name="line.222"></a>
-<span class="sourceLineNo">223</span>        if 
(region.getRegionInfo().isSystemTable()) {<a name="line.223"></a>
-<span class="sourceLineNo">224</span>          if (LOG.isTraceEnabled()) {<a 
name="line.224"></a>
-<span class="sourceLineNo">225</span>            LOG.trace("High priority 
because region=" +<a name="line.225"></a>
-<span class="sourceLineNo">226</span>              
region.getRegionInfo().getRegionNameAsString());<a name="line.226"></a>
-<span class="sourceLineNo">227</span>          }<a name="line.227"></a>
-<span class="sourceLineNo">228</span>          return 
HConstants.SYSTEMTABLE_QOS;<a name="line.228"></a>
-<span class="sourceLineNo">229</span>        }<a name="line.229"></a>
-<span class="sourceLineNo">230</span>      }<a name="line.230"></a>
-<span class="sourceLineNo">231</span>    } catch (Exception ex) {<a 
name="line.231"></a>
-<span class="sourceLineNo">232</span>      // Not good throwing an exception 
out of here, a runtime anyways.  Let the query go into the<a 
name="line.232"></a>
-<span class="sourceLineNo">233</span>      // server and have it throw the 
exception if still an issue.  Just mark it normal priority.<a 
name="line.233"></a>
-<span class="sourceLineNo">234</span>      if (LOG.isTraceEnabled()) 
LOG.trace("Marking normal priority after getting exception=" + ex);<a 
name="line.234"></a>
-<span class="sourceLineNo">235</span>      return HConstants.NORMAL_QOS;<a 
name="line.235"></a>
-<span class="sourceLineNo">236</span>    }<a name="line.236"></a>
-<span class="sourceLineNo">237</span><a name="line.237"></a>
-<span class="sourceLineNo">238</span>    if (param instanceof ScanRequest) { 
// scanner methods...<a name="line.238"></a>
-<span class="sourceLineNo">239</span>      ScanRequest request = 
(ScanRequest)param;<a name="line.239"></a>
-<span class="sourceLineNo">240</span>      if (!request.hasScannerId()) {<a 
name="line.240"></a>
-<span class="sourceLineNo">241</span>        return HConstants.NORMAL_QOS;<a 
name="line.241"></a>
-<span class="sourceLineNo">242</span>      }<a name="line.242"></a>
-<span class="sourceLineNo">243</span>      RegionScanner scanner = 
rpcServices.getScanner(request.getScannerId());<a name="line.243"></a>
-<span class="sourceLineNo">244</span>      if (scanner != null &amp;&amp; 
scanner.getRegionInfo().isSystemTable()) {<a name="line.244"></a>
-<span class="sourceLineNo">245</span>        if (LOG.isTraceEnabled()) {<a 
name="line.245"></a>
-<span class="sourceLineNo">246</span>          // Scanner requests are small 
in size so TextFormat version should not overwhelm log.<a name="line.246"></a>
-<span class="sourceLineNo">247</span>          LOG.trace("High priority 
scanner request " + TextFormat.shortDebugString(request));<a 
name="line.247"></a>
-<span class="sourceLineNo">248</span>        }<a name="line.248"></a>
-<span class="sourceLineNo">249</span>        return 
HConstants.SYSTEMTABLE_QOS;<a name="line.249"></a>
-<span class="sourceLineNo">250</span>      }<a name="line.250"></a>
-<span class="sourceLineNo">251</span>    }<a name="line.251"></a>
-<span class="sourceLineNo">252</span><a name="line.252"></a>
-<span class="sourceLineNo">253</span>    return HConstants.NORMAL_QOS;<a 
name="line.253"></a>
-<span class="sourceLineNo">254</span>  }<a name="line.254"></a>
-<span class="sourceLineNo">255</span><a name="line.255"></a>
-<span class="sourceLineNo">256</span>  /**<a name="line.256"></a>
-<span class="sourceLineNo">257</span>   * Based on the request content, 
returns the deadline of the request.<a name="line.257"></a>
-<span class="sourceLineNo">258</span>   *<a name="line.258"></a>
-<span class="sourceLineNo">259</span>   * @param header<a name="line.259"></a>
-<span class="sourceLineNo">260</span>   * @param param<a name="line.260"></a>
-<span class="sourceLineNo">261</span>   * @return Deadline of this request. 0 
now, otherwise msec of 'delay'<a name="line.261"></a>
-<span class="sourceLineNo">262</span>   */<a name="line.262"></a>
-<span class="sourceLineNo">263</span>  @Override<a name="line.263"></a>
-<span class="sourceLineNo">264</span>  public long getDeadline(RequestHeader 
header, Message param) {<a name="line.264"></a>
-<span class="sourceLineNo">265</span>    if (param instanceof ScanRequest) {<a 
name="line.265"></a>
-<span class="sourceLineNo">266</span>      ScanRequest request = 
(ScanRequest)param;<a name="line.266"></a>
-<span class="sourceLineNo">267</span>      if (!request.hasScannerId()) {<a 
name="line.267"></a>
-<span class="sourceLineNo">268</span>        return 0;<a name="line.268"></a>
-<span class="sourceLineNo">269</span>      }<a name="line.269"></a>
-<span class="sourceLineNo">270</span><a name="line.270"></a>
-<span class="sourceLineNo">271</span>      // get the 'virtual time' of the 
scanner, and applies sqrt() to get a<a name="line.271"></a>
-<span class="sourceLineNo">272</span>      // nice curve for the delay. More a 
scanner is used the less priority it gets.<a name="line.272"></a>
-<span class="sourceLineNo">273</span>      // The weight is used to have more 
control on the delay.<a name="line.273"></a>
-<span class="sourceLineNo">274</span>      long vtime = 
rpcServices.getScannerVirtualTime(request.getScannerId());<a 
name="line.274"></a>
-<span class="sourceLineNo">275</span>      return Math.round(Math.sqrt(vtime * 
scanVirtualTimeWeight));<a name="line.275"></a>
-<span class="sourceLineNo">276</span>    }<a name="line.276"></a>
-<span class="sourceLineNo">277</span>    return 0;<a name="line.277"></a>
-<span class="sourceLineNo">278</span>  }<a name="line.278"></a>
-<span class="sourceLineNo">279</span><a name="line.279"></a>
-<span class="sourceLineNo">280</span>  @VisibleForTesting<a 
name="line.280"></a>
-<span class="sourceLineNo">281</span>  void setRegionServer(final 
HRegionServer hrs) {<a name="line.281"></a>
-<span class="sourceLineNo">282</span>    this.rpcServices = 
hrs.getRSRpcServices();<a name="line.282"></a>
-<span class="sourceLineNo">283</span>  }<a name="line.283"></a>
-<span class="sourceLineNo">284</span>}<a name="line.284"></a>
+<span class="sourceLineNo">098</span>  /**<a name="line.98"></a>
+<span class="sourceLineNo">099</span>   * Calls {@link 
#AnnotationReadingPriorityFunction(RSRpcServices, Class)} using the result of<a 
name="line.99"></a>
+<span class="sourceLineNo">100</span>   * {@code rpcServices#getClass()}<a 
name="line.100"></a>
+<span class="sourceLineNo">101</span>   *<a name="line.101"></a>
+<span class="sourceLineNo">102</span>   * @param rpcServices<a 
name="line.102"></a>
+<span class="sourceLineNo">103</span>   *          The RPC server 
implementation<a name="line.103"></a>
+<span class="sourceLineNo">104</span>   */<a name="line.104"></a>
+<span class="sourceLineNo">105</span>  public 
AnnotationReadingPriorityFunction(final RSRpcServices rpcServices) {<a 
name="line.105"></a>
+<span class="sourceLineNo">106</span>    this(rpcServices, 
rpcServices.getClass());<a name="line.106"></a>
+<span class="sourceLineNo">107</span>  }<a name="line.107"></a>
+<span class="sourceLineNo">108</span><a name="line.108"></a>
+<span class="sourceLineNo">109</span>  /**<a name="line.109"></a>
+<span class="sourceLineNo">110</span>   * Constructs the priority function 
given the RPC server implementation and the annotations on the<a 
name="line.110"></a>
+<span class="sourceLineNo">111</span>   * methods in the provided {@code 
clz}.<a name="line.111"></a>
+<span class="sourceLineNo">112</span>   *<a name="line.112"></a>
+<span class="sourceLineNo">113</span>   * @param rpcServices<a 
name="line.113"></a>
+<span class="sourceLineNo">114</span>   *          The RPC server 
implementation<a name="line.114"></a>
+<span class="sourceLineNo">115</span>   * @param clz<a name="line.115"></a>
+<span class="sourceLineNo">116</span>   *          The concrete RPC server 
implementation's class<a name="line.116"></a>
+<span class="sourceLineNo">117</span>   */<a name="line.117"></a>
+<span class="sourceLineNo">118</span>  public 
AnnotationReadingPriorityFunction(final RSRpcServices rpcServices,<a 
name="line.118"></a>
+<span class="sourceLineNo">119</span>      Class&lt;? extends 
RSRpcServices&gt; clz) {<a name="line.119"></a>
+<span class="sourceLineNo">120</span>    Map&lt;String,Integer&gt; qosMap = 
new HashMap&lt;&gt;();<a name="line.120"></a>
+<span class="sourceLineNo">121</span>    for (Method m : clz.getMethods()) {<a 
name="line.121"></a>
+<span class="sourceLineNo">122</span>      QosPriority p = 
m.getAnnotation(QosPriority.class);<a name="line.122"></a>
+<span class="sourceLineNo">123</span>      if (p != null) {<a 
name="line.123"></a>
+<span class="sourceLineNo">124</span>        // Since we protobuf'd, and then 
subsequently, when we went with pb style, method names<a name="line.124"></a>
+<span class="sourceLineNo">125</span>        // are capitalized.  This meant 
that this brittle compare of method names gotten by<a name="line.125"></a>
+<span class="sourceLineNo">126</span>        // reflection no longer matched 
the method names coming in over pb.  TODO: Get rid of this<a 
name="line.126"></a>
+<span class="sourceLineNo">127</span>        // check.  For now, workaround is 
to capitalize the names we got from reflection so they<a name="line.127"></a>
+<span class="sourceLineNo">128</span>        // have chance of matching the pb 
ones.<a name="line.128"></a>
+<span class="sourceLineNo">129</span>        String capitalizedMethodName = 
capitalize(m.getName());<a name="line.129"></a>
+<span class="sourceLineNo">130</span>        qosMap.put(capitalizedMethodName, 
p.priority());<a name="line.130"></a>
+<span class="sourceLineNo">131</span>      }<a name="line.131"></a>
+<span class="sourceLineNo">132</span>    }<a name="line.132"></a>
+<span class="sourceLineNo">133</span>    this.rpcServices = rpcServices;<a 
name="line.133"></a>
+<span class="sourceLineNo">134</span>    this.annotatedQos = qosMap;<a 
name="line.134"></a>
+<span class="sourceLineNo">135</span>    if (methodMap.get("getRegion") == 
null) {<a name="line.135"></a>
+<span class="sourceLineNo">136</span>      methodMap.put("hasRegion", new 
HashMap&lt;&gt;());<a name="line.136"></a>
+<span class="sourceLineNo">137</span>      methodMap.put("getRegion", new 
HashMap&lt;&gt;());<a name="line.137"></a>
+<span class="sourceLineNo">138</span>    }<a name="line.138"></a>
+<span class="sourceLineNo">139</span>    for (Class&lt;? extends Message&gt; 
cls : knownArgumentClasses) {<a name="line.139"></a>
+<span class="sourceLineNo">140</span>      
argumentToClassMap.put(cls.getName(), cls);<a name="line.140"></a>
+<span class="sourceLineNo">141</span>      try {<a name="line.141"></a>
+<span class="sourceLineNo">142</span>        
methodMap.get("hasRegion").put(cls, cls.getDeclaredMethod("hasRegion"));<a 
name="line.142"></a>
+<span class="sourceLineNo">143</span>        
methodMap.get("getRegion").put(cls, cls.getDeclaredMethod("getRegion"));<a 
name="line.143"></a>
+<span class="sourceLineNo">144</span>      } catch (Exception e) {<a 
name="line.144"></a>
+<span class="sourceLineNo">145</span>        throw new RuntimeException(e);<a 
name="line.145"></a>
+<span class="sourceLineNo">146</span>      }<a name="line.146"></a>
+<span class="sourceLineNo">147</span>    }<a name="line.147"></a>
+<span class="sourceLineNo">148</span><a name="line.148"></a>
+<span class="sourceLineNo">149</span>    Configuration conf = 
rpcServices.getConfiguration();<a name="line.149"></a>
+<span class="sourceLineNo">150</span>    scanVirtualTimeWeight = 
conf.getFloat(SCAN_VTIME_WEIGHT_CONF_KEY, 1.0f);<a name="line.150"></a>
+<span class="sourceLineNo">151</span>  }<a name="line.151"></a>
+<span class="sourceLineNo">152</span><a name="line.152"></a>
+<span class="sourceLineNo">153</span>  private String capitalize(final String 
s) {<a name="line.153"></a>
+<span class="sourceLineNo">154</span>    StringBuilder strBuilder = new 
StringBuilder(s);<a name="line.154"></a>
+<span class="sourceLineNo">155</span>    strBuilder.setCharAt(0, 
Character.toUpperCase(strBuilder.charAt(0)));<a name="line.155"></a>
+<span class="sourceLineNo">156</span>    return strBuilder.toString();<a 
name="line.156"></a>
+<span class="sourceLineNo">157</span>  }<a name="line.157"></a>
+<span class="sourceLineNo">158</span><a name="line.158"></a>
+<span class="sourceLineNo">159</span>  /**<a name="line.159"></a>
+<span class="sourceLineNo">160</span>   * Returns a 'priority' based on the 
request type.<a name="line.160"></a>
+<span class="sourceLineNo">161</span>   *<a name="line.161"></a>
+<span class="sourceLineNo">162</span>   * Currently the returned priority is 
used for queue selection.<a name="line.162"></a>
+<span class="sourceLineNo">163</span>   * See the SimpleRpcScheduler as 
example. It maintains a queue per 'priory type'<a name="line.163"></a>
+<span class="sourceLineNo">164</span>   * HIGH_QOS (meta requests), 
REPLICATION_QOS (replication requests),<a name="line.164"></a>
+<span class="sourceLineNo">165</span>   * NORMAL_QOS (user requests).<a 
name="line.165"></a>
+<span class="sourceLineNo">166</span>   */<a name="line.166"></a>
+<span class="sourceLineNo">167</span>  @Override<a name="line.167"></a>
+<span class="sourceLineNo">168</span>  public int getPriority(RequestHeader 
header, Message param, User user) {<a name="line.168"></a>
+<span class="sourceLineNo">169</span>    int priorityByAnnotation = 
getAnnotatedPriority(header);<a name="line.169"></a>
+<span class="sourceLineNo">170</span><a name="line.170"></a>
+<span class="sourceLineNo">171</span>    if (priorityByAnnotation &gt;= 0) {<a 
name="line.171"></a>
+<span class="sourceLineNo">172</span>      return priorityByAnnotation;<a 
name="line.172"></a>
+<span class="sourceLineNo">173</span>    }<a name="line.173"></a>
+<span class="sourceLineNo">174</span>    return getBasePriority(header, 
param);<a name="line.174"></a>
+<span class="sourceLineNo">175</span>  }<a name="line.175"></a>
+<span class="sourceLineNo">176</span><a name="line.176"></a>
+<span class="sourceLineNo">177</span>  /**<a name="line.177"></a>
+<span class="sourceLineNo">178</span>   * See if the method has an 
annotation.<a name="line.178"></a>
+<span class="sourceLineNo">179</span>   * @param header<a name="line.179"></a>
+<span class="sourceLineNo">180</span>   * @return Return the priority from the 
annotation. If there isn't<a name="line.180"></a>
+<span class="sourceLineNo">181</span>   * an annotation, this returns 
something below zero.<a name="line.181"></a>
+<span class="sourceLineNo">182</span>   */<a name="line.182"></a>
+<span class="sourceLineNo">183</span>  protected int 
getAnnotatedPriority(RequestHeader header) {<a name="line.183"></a>
+<span class="sourceLineNo">184</span>    String methodName = 
header.getMethodName();<a name="line.184"></a>
+<span class="sourceLineNo">185</span>    Integer priorityByAnnotation = 
annotatedQos.get(methodName);<a name="line.185"></a>
+<span class="sourceLineNo">186</span>    if (priorityByAnnotation != null) {<a 
name="line.186"></a>
+<span class="sourceLineNo">187</span>      return priorityByAnnotation;<a 
name="line.187"></a>
+<span class="sourceLineNo">188</span>    }<a name="line.188"></a>
+<span class="sourceLineNo">189</span>    return -1;<a name="line.189"></a>
+<span class="sourceLineNo">190</span>  }<a name="line.190"></a>
+<span class="sourceLineNo">191</span><a name="line.191"></a>
+<span class="sourceLineNo">192</span>  /**<a name="line.192"></a>
+<span class="sourceLineNo">193</span>   * Get the priority for a given request 
from the header and the param<a name="line.193"></a>
+<span class="sourceLineNo">194</span>   * This doesn't consider which user is 
sending the request at all.<a name="line.194"></a>
+<span class="sourceLineNo">195</span>   * This doesn't consider annotations<a 
name="line.195"></a>
+<span class="sourceLineNo">196</span>   */<a name="line.196"></a>
+<span class="sourceLineNo">197</span>  protected int 
getBasePriority(RequestHeader header, Message param) {<a name="line.197"></a>
+<span class="sourceLineNo">198</span>    if (param == null) {<a 
name="line.198"></a>
+<span class="sourceLineNo">199</span>      return HConstants.NORMAL_QOS;<a 
name="line.199"></a>
+<span class="sourceLineNo">200</span>    }<a name="line.200"></a>
+<span class="sourceLineNo">201</span><a name="line.201"></a>
+<span class="sourceLineNo">202</span>    // Trust the client-set priorities if 
set<a name="line.202"></a>
+<span class="sourceLineNo">203</span>    if (header.hasPriority()) {<a 
name="line.203"></a>
+<span class="sourceLineNo">204</span>      return header.getPriority();<a 
name="line.204"></a>
+<span class="sourceLineNo">205</span>    }<a name="line.205"></a>
+<span class="sourceLineNo">206</span><a name="line.206"></a>
+<span class="sourceLineNo">207</span>    String cls = 
param.getClass().getName();<a name="line.207"></a>
+<span class="sourceLineNo">208</span>    Class&lt;? extends Message&gt; 
rpcArgClass = argumentToClassMap.get(cls);<a name="line.208"></a>
+<span class="sourceLineNo">209</span>    RegionSpecifier regionSpecifier = 
null;<a name="line.209"></a>
+<span class="sourceLineNo">210</span>    //check whether the request has 
reference to meta region or now.<a name="line.210"></a>
+<span class="sourceLineNo">211</span>    try {<a name="line.211"></a>
+<span class="sourceLineNo">212</span>      // Check if the param has a region 
specifier; the pb methods are hasRegion and getRegion if<a name="line.212"></a>
+<span class="sourceLineNo">213</span>      // hasRegion returns true.  Not all 
listed methods have region specifier each time.  For<a name="line.213"></a>
+<span class="sourceLineNo">214</span>      // example, the ScanRequest has it 
on setup but thereafter relies on the scannerid rather than<a 
name="line.214"></a>
+<span class="sourceLineNo">215</span>      // send the region over every 
time.<a name="line.215"></a>
+<span class="sourceLineNo">216</span>      Method hasRegion = 
methodMap.get("hasRegion").get(rpcArgClass);<a name="line.216"></a>
+<span class="sourceLineNo">217</span>      if (hasRegion != null &amp;&amp; 
(Boolean)hasRegion.invoke(param, (Object[])null)) {<a name="line.217"></a>
+<span class="sourceLineNo">218</span>        Method getRegion = 
methodMap.get("getRegion").get(rpcArgClass);<a name="line.218"></a>
+<span class="sourceLineNo">219</span>        regionSpecifier = 
(RegionSpecifier)getRegion.invoke(param, (Object[])null);<a name="line.219"></a>
+<span class="sourceLineNo">220</span>        Region region = 
rpcServices.getRegion(regionSpecifier);<a name="line.220"></a>
+<span class="sourceLineNo">221</span>        if 
(region.getRegionInfo().isSystemTable()) {<a name="line.221"></a>
+<span class="sourceLineNo">222</span>          if (LOG.isTraceEnabled()) {<a 
name="line.222"></a>
+<span class="sourceLineNo">223</span>            LOG.trace("High priority 
because region=" +<a name="line.223"></a>
+<span class="sourceLineNo">224</span>              
region.getRegionInfo().getRegionNameAsString());<a name="line.224"></a>
+<span class="sourceLineNo">225</span>          }<a name="line.225"></a>
+<span class="sourceLineNo">226</span>          return 
HConstants.SYSTEMTABLE_QOS;<a name="line.226"></a>
+<span class="sourceLineNo">227</span>        }<a name="line.227"></a>
+<span class="sourceLineNo">228</span>      }<a name="line.228"></a>
+<span class="sourceLineNo">229</span>    } catch (Exception ex) {<a 
name="line.229"></a>
+<span class="sourceLineNo">230</span>      // Not good throwing an exception 
out of here, a runtime anyways.  Let the query go into the<a 
name="line.230"></a>
+<span class="sourceLineNo">231</span>      // server and have it throw the 
exception if still an issue.  Just mark it normal priority.<a 
name="line.231"></a>
+<span class="sourceLineNo">232</span>      if (LOG.isTraceEnabled()) 
LOG.trace("Marking normal priority after getting exception=" + ex);<a 
name="line.232"></a>
+<span class="sourceLineNo">233</span>      return HConstants.NORMAL_QOS;<a 
name="line.233"></a>
+<span class="sourceLineNo">234</span>    }<a name="line.234"></a>
+<span class="sourceLineNo">235</span><a name="line.235"></a>
+<span class="sourceLineNo">236</span>    if (param instanceof ScanRequest) { 
// scanner methods...<a name="line.236"></a>
+<span class="sourceLineNo">237</span>      ScanRequest request = 
(ScanRequest)param;<a name="line.237"></a>
+<span class="sourceLineNo">238</span>      if (!request.hasScannerId()) {<a 
name="line.238"></a>
+<span class="sourceLineNo">239</span>        return HConstants.NORMAL_QOS;<a 
name="line.239"></a>
+<span class="sourceLineNo">240</span>      }<a name="line.240"></a>
+<span class="sourceLineNo">241</span>      RegionScanner scanner = 
rpcServices.getScanner(request.getScannerId());<a name="line.241"></a>
+<span class="sourceLineNo">242</span>      if (scanner != null &amp;&amp; 
scanner.getRegionInfo().isSystemTable()) {<a name="line.242"></a>
+<span class="sourceLineNo">243</span>        if (LOG.isTraceEnabled()) {<a 
name="line.243"></a>
+<span class="sourceLineNo">244</span>          // Scanner requests are small 
in size so TextFormat version should not overwhelm log.<a name="line.244"></a>
+<span class="sourceLineNo">245</span>          LOG.trace("High priority 
scanner request " + TextFormat.shortDebugString(request));<a 
name="line.245"></a>
+<span class="sourceLineNo">246</span>        }<a name="line.246"></a>
+<span class="sourceLineNo">247</span>        return 
HConstants.SYSTEMTABLE_QOS;<a name="line.247"></a>
+<span class="sourceLineNo">248</span>      }<a name="line.248"></a>
+<span class="sourceLineNo">249</span>    }<a name="line.249"></a>
+<span class="sourceLineNo">250</span><a name="line.250"></a>
+<span class="sourceLineNo">251</span>    return HConstants.NORMAL_QOS;<a 
name="line.251"></a>
+<span class="sourceLineNo">252</span>  }<a name="line.252"></a>
+<span class="sourceLineNo">253</span><a name="line.253"></a>
+<span class="sourceLineNo">254</span>  /**<a name="line.254"></a>
+<span class="sourceLineNo">255</span>   * Based on the request content, 
returns the deadline of the request.<a name="line.255"></a>
+<span class="sourceLineNo">256</span>   *<a name="line.256"></a>
+<span class="sourceLineNo">257</span>   * @param header<a name="line.257"></a>
+<span class="sourceLineNo">258</span>   * @param param<a name="line.258"></a>
+<span class="sourceLineNo">259</span>   * @return Deadline of this request. 0 
now, otherwise msec of 'delay'<a name="line.259"></a>
+<span class="sourceLineNo">260</span>   */<a name="line.260"></a>
+<span class="sourceLineNo">261</span>  @Override<a name="line.261"></a>
+<span class="sourceLineNo">262</span>  public long getDeadline(RequestHeader 
header, Message param) {<a name="line.262"></a>
+<span class="sourceLineNo">263</span>    if (param instanceof ScanRequest) {<a 
name="line.263"></a>
+<span class="sourceLineNo">264</span>      ScanRequest request = 
(ScanRequest)param;<a name="line.264"></a>
+<span class="sourceLineNo">265</span>      if (!request.hasScannerId()) {<a 
name="line.265"></a>
+<span class="sourceLineNo">266</span>        return 0;<a name="line.266"></a>
+<span class="sourceLineNo">267</span>      }<a name="line.267"></a>
+<span class="sourceLineNo">268</span><a name="line.268"></a>
+<span class="sourceLineNo">269</span>      // get the 'virtual time' of the 
scanner, and applies sqrt() to get a<a name="line.269"></a>
+<span class="sourceLineNo">270</span>      // nice curve for the delay. More a 
scanner is used the less priority it gets.<a name="line.270"></a>
+<span class="sourceLineNo">271</span>      // The weight is used to have more 
control on the delay.<a name="line.271"></a>
+<span class="sourceLineNo">272</span>      long vtime = 
rpcServices.getScannerVirtualTime(request.getScannerId());<a 
name="line.272"></a>
+<span class="sourceLineNo">273</span>      return Math.round(Math.sqrt(vtime * 
scanVirtualTimeWeight));<a name="line.273"></a>
+<span class="sourceLineNo">274</span>    }<a name="line.274"></a>
+<span class="sourceLineNo">275</span>    return 0;<a name="line.275"></a>
+<span class="sourceLineNo">276</span>  }<a name="line.276"></a>
+<span class="sourceLineNo">277</span><a name="line.277"></a>
+<span class="sourceLineNo">278</span>  @VisibleForTesting<a 
name="line.278"></a>
+<span class="sourceLineNo">279</span>  void setRegionServer(final 
HRegionServer hrs) {<a name="line.279"></a>
+<span class="sourceLineNo">280</span>    this.rpcServices = 
hrs.getRSRpcServices();<a name="line.280"></a>
+<span class="sourceLineNo">281</span>  }<a name="line.281"></a>
+<span class="sourceLineNo">282</span>}<a name="line.282"></a>
 
 
 

http://git-wip-us.apache.org/repos/asf/hbase-site/blob/1837997e/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/MetricsRegionServer.html
----------------------------------------------------------------------
diff --git 
a/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/MetricsRegionServer.html
 
b/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/MetricsRegionServer.html
index be63802..288ab85 100644
--- 
a/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/MetricsRegionServer.html
+++ 
b/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/MetricsRegionServer.html
@@ -79,84 +79,100 @@
 <span class="sourceLineNo">071</span>    return regionServerWrapper;<a 
name="line.71"></a>
 <span class="sourceLineNo">072</span>  }<a name="line.72"></a>
 <span class="sourceLineNo">073</span><a name="line.73"></a>
-<span class="sourceLineNo">074</span>  public void updatePut(long t) {<a 
name="line.74"></a>
+<span class="sourceLineNo">074</span>  public void updatePutBatch(long t) {<a 
name="line.74"></a>
 <span class="sourceLineNo">075</span>    if (t &gt; 1000) {<a 
name="line.75"></a>
 <span class="sourceLineNo">076</span>      serverSource.incrSlowPut();<a 
name="line.76"></a>
 <span class="sourceLineNo">077</span>    }<a name="line.77"></a>
-<span class="sourceLineNo">078</span>    serverSource.updatePut(t);<a 
name="line.78"></a>
+<span class="sourceLineNo">078</span>    serverSource.updatePutBatch(t);<a 
name="line.78"></a>
 <span class="sourceLineNo">079</span>  }<a name="line.79"></a>
 <span class="sourceLineNo">080</span><a name="line.80"></a>
-<span class="sourceLineNo">081</span>  public void updateDelete(long t) {<a 
name="line.81"></a>
-<span class="sourceLineNo">082</span>    if (t &gt; 1000) {<a 
name="line.82"></a>
-<span class="sourceLineNo">083</span>      serverSource.incrSlowDelete();<a 
name="line.83"></a>
-<span class="sourceLineNo">084</span>    }<a name="line.84"></a>
-<span class="sourceLineNo">085</span>    serverSource.updateDelete(t);<a 
name="line.85"></a>
-<span class="sourceLineNo">086</span>  }<a name="line.86"></a>
-<span class="sourceLineNo">087</span><a name="line.87"></a>
-<span class="sourceLineNo">088</span>  public void updateGet(long t) {<a 
name="line.88"></a>
-<span class="sourceLineNo">089</span>    if (t &gt; 1000) {<a 
name="line.89"></a>
-<span class="sourceLineNo">090</span>      serverSource.incrSlowGet();<a 
name="line.90"></a>
-<span class="sourceLineNo">091</span>    }<a name="line.91"></a>
-<span class="sourceLineNo">092</span>    serverSource.updateGet(t);<a 
name="line.92"></a>
-<span class="sourceLineNo">093</span>  }<a name="line.93"></a>
-<span class="sourceLineNo">094</span><a name="line.94"></a>
-<span class="sourceLineNo">095</span>  public void updateIncrement(long t) {<a 
name="line.95"></a>
-<span class="sourceLineNo">096</span>    if (t &gt; 1000) {<a 
name="line.96"></a>
-<span class="sourceLineNo">097</span>      serverSource.incrSlowIncrement();<a 
name="line.97"></a>
-<span class="sourceLineNo">098</span>    }<a name="line.98"></a>
-<span class="sourceLineNo">099</span>    serverSource.updateIncrement(t);<a 
name="line.99"></a>
-<span class="sourceLineNo">100</span>  }<a name="line.100"></a>
-<span class="sourceLineNo">101</span><a name="line.101"></a>
-<span class="sourceLineNo">102</span>  public void updateAppend(long t) {<a 
name="line.102"></a>
-<span class="sourceLineNo">103</span>    if (t &gt; 1000) {<a 
name="line.103"></a>
-<span class="sourceLineNo">104</span>      serverSource.incrSlowAppend();<a 
name="line.104"></a>
-<span class="sourceLineNo">105</span>    }<a name="line.105"></a>
-<span class="sourceLineNo">106</span>    serverSource.updateAppend(t);<a 
name="line.106"></a>
-<span class="sourceLineNo">107</span>  }<a name="line.107"></a>
-<span class="sourceLineNo">108</span><a name="line.108"></a>
-<span class="sourceLineNo">109</span>  public void updateReplay(long t){<a 
name="line.109"></a>
-<span class="sourceLineNo">110</span>    serverSource.updateReplay(t);<a 
name="line.110"></a>
-<span class="sourceLineNo">111</span>  }<a name="line.111"></a>
-<span class="sourceLineNo">112</span><a name="line.112"></a>
-<span class="sourceLineNo">113</span>  public void updateScanSize(long 
scanSize){<a name="line.113"></a>
-<span class="sourceLineNo">114</span>    
serverSource.updateScanSize(scanSize);<a name="line.114"></a>
-<span class="sourceLineNo">115</span>  }<a name="line.115"></a>
-<span class="sourceLineNo">116</span><a name="line.116"></a>
-<span class="sourceLineNo">117</span>  public void updateScanTime(long t) {<a 
name="line.117"></a>
-<span class="sourceLineNo">118</span>    serverSource.updateScanTime(t);<a 
name="line.118"></a>
-<span class="sourceLineNo">119</span>  }<a name="line.119"></a>
-<span class="sourceLineNo">120</span><a name="line.120"></a>
-<span class="sourceLineNo">121</span>  public void updateSplitTime(long t) {<a 
name="line.121"></a>
-<span class="sourceLineNo">122</span>    serverSource.updateSplitTime(t);<a 
name="line.122"></a>
+<span class="sourceLineNo">081</span>  public void updatePut(long t) {<a 
name="line.81"></a>
+<span class="sourceLineNo">082</span>    serverSource.updatePut(t);<a 
name="line.82"></a>
+<span class="sourceLineNo">083</span>  }<a name="line.83"></a>
+<span class="sourceLineNo">084</span><a name="line.84"></a>
+<span class="sourceLineNo">085</span>  public void updateDelete(long t) {<a 
name="line.85"></a>
+<span class="sourceLineNo">086</span>    serverSource.updateDelete(t);<a 
name="line.86"></a>
+<span class="sourceLineNo">087</span>  }<a name="line.87"></a>
+<span class="sourceLineNo">088</span><a name="line.88"></a>
+<span class="sourceLineNo">089</span>  public void updateDeleteBatch(long t) 
{<a name="line.89"></a>
+<span class="sourceLineNo">090</span>    if (t &gt; 1000) {<a 
name="line.90"></a>
+<span class="sourceLineNo">091</span>      serverSource.incrSlowDelete();<a 
name="line.91"></a>
+<span class="sourceLineNo">092</span>    }<a name="line.92"></a>
+<span class="sourceLineNo">093</span>    serverSource.updateDeleteBatch(t);<a 
name="line.93"></a>
+<span class="sourceLineNo">094</span>  }<a name="line.94"></a>
+<span class="sourceLineNo">095</span><a name="line.95"></a>
+<span class="sourceLineNo">096</span>  public void updateCheckAndDelete(long 
t) {<a name="line.96"></a>
+<span class="sourceLineNo">097</span>    
serverSource.updateCheckAndDelete(t);<a name="line.97"></a>
+<span class="sourceLineNo">098</span>  }<a name="line.98"></a>
+<span class="sourceLineNo">099</span><a name="line.99"></a>
+<span class="sourceLineNo">100</span>  public void updateCheckAndPut(long t) 
{<a name="line.100"></a>
+<span class="sourceLineNo">101</span>    serverSource.updateCheckAndPut(t);<a 
name="line.101"></a>
+<span class="sourceLineNo">102</span>  }<a name="line.102"></a>
+<span class="sourceLineNo">103</span><a name="line.103"></a>
+<span class="sourceLineNo">104</span>  public void updateGet(long t) {<a 
name="line.104"></a>
+<span class="sourceLineNo">105</span>    if (t &gt; 1000) {<a 
name="line.105"></a>
+<span class="sourceLineNo">106</span>      serverSource.incrSlowGet();<a 
name="line.106"></a>
+<span class="sourceLineNo">107</span>    }<a name="line.107"></a>
+<span class="sourceLineNo">108</span>    serverSource.updateGet(t);<a 
name="line.108"></a>
+<span class="sourceLineNo">109</span>  }<a name="line.109"></a>
+<span class="sourceLineNo">110</span><a name="line.110"></a>
+<span class="sourceLineNo">111</span>  public void updateIncrement(long t) {<a 
name="line.111"></a>
+<span class="sourceLineNo">112</span>    if (t &gt; 1000) {<a 
name="line.112"></a>
+<span class="sourceLineNo">113</span>      serverSource.incrSlowIncrement();<a 
name="line.113"></a>
+<span class="sourceLineNo">114</span>    }<a name="line.114"></a>
+<span class="sourceLineNo">115</span>    serverSource.updateIncrement(t);<a 
name="line.115"></a>
+<span class="sourceLineNo">116</span>  }<a name="line.116"></a>
+<span class="sourceLineNo">117</span><a name="line.117"></a>
+<span class="sourceLineNo">118</span>  public void updateAppend(long t) {<a 
name="line.118"></a>
+<span class="sourceLineNo">119</span>    if (t &gt; 1000) {<a 
name="line.119"></a>
+<span class="sourceLineNo">120</span>      serverSource.incrSlowAppend();<a 
name="line.120"></a>
+<span class="sourceLineNo">121</span>    }<a name="line.121"></a>
+<span class="sourceLineNo">122</span>    serverSource.updateAppend(t);<a 
name="line.122"></a>
 <span class="sourceLineNo">123</span>  }<a name="line.123"></a>
 <span class="sourceLineNo">124</span><a name="line.124"></a>
-<span class="sourceLineNo">125</span>  public void incrSplitRequest() {<a 
name="line.125"></a>
-<span class="sourceLineNo">126</span>    serverSource.incrSplitRequest();<a 
name="line.126"></a>
+<span class="sourceLineNo">125</span>  public void updateReplay(long t){<a 
name="line.125"></a>
+<span class="sourceLineNo">126</span>    serverSource.updateReplay(t);<a 
name="line.126"></a>
 <span class="sourceLineNo">127</span>  }<a name="line.127"></a>
 <span class="sourceLineNo">128</span><a name="line.128"></a>
-<span class="sourceLineNo">129</span>  public void incrSplitSuccess() {<a 
name="line.129"></a>
-<span class="sourceLineNo">130</span>    serverSource.incrSplitSuccess();<a 
name="line.130"></a>
+<span class="sourceLineNo">129</span>  public void updateScanSize(long 
scanSize){<a name="line.129"></a>
+<span class="sourceLineNo">130</span>    
serverSource.updateScanSize(scanSize);<a name="line.130"></a>
 <span class="sourceLineNo">131</span>  }<a name="line.131"></a>
 <span class="sourceLineNo">132</span><a name="line.132"></a>
-<span class="sourceLineNo">133</span>  public void updateFlush(long t, long 
memstoreSize, long fileSize) {<a name="line.133"></a>
-<span class="sourceLineNo">134</span>    serverSource.updateFlushTime(t);<a 
name="line.134"></a>
-<span class="sourceLineNo">135</span>    
serverSource.updateFlushMemstoreSize(memstoreSize);<a name="line.135"></a>
-<span class="sourceLineNo">136</span>    
serverSource.updateFlushOutputSize(fileSize);<a name="line.136"></a>
-<span class="sourceLineNo">137</span>  }<a name="line.137"></a>
-<span class="sourceLineNo">138</span><a name="line.138"></a>
-<span class="sourceLineNo">139</span>  public void updateCompaction(boolean 
isMajor, long t, int inputFileCount, int outputFileCount,<a name="line.139"></a>
-<span class="sourceLineNo">140</span>      long inputBytes, long outputBytes) 
{<a name="line.140"></a>
-<span class="sourceLineNo">141</span>    
serverSource.updateCompactionTime(isMajor, t);<a name="line.141"></a>
-<span class="sourceLineNo">142</span>    
serverSource.updateCompactionInputFileCount(isMajor, inputFileCount);<a 
name="line.142"></a>
-<span class="sourceLineNo">143</span>    
serverSource.updateCompactionOutputFileCount(isMajor, outputFileCount);<a 
name="line.143"></a>
-<span class="sourceLineNo">144</span>    
serverSource.updateCompactionInputSize(isMajor, inputBytes);<a 
name="line.144"></a>
-<span class="sourceLineNo">145</span>    
serverSource.updateCompactionOutputSize(isMajor, outputBytes);<a 
name="line.145"></a>
-<span class="sourceLineNo">146</span>  }<a name="line.146"></a>
-<span class="sourceLineNo">147</span><a name="line.147"></a>
-<span class="sourceLineNo">148</span>  public void updateBulkLoad(long millis) 
{<a name="line.148"></a>
-<span class="sourceLineNo">149</span>    
this.bulkLoadTimer.updateMillis(millis);<a name="line.149"></a>
-<span class="sourceLineNo">150</span>  }<a name="line.150"></a>
-<span class="sourceLineNo">151</span>}<a name="line.151"></a>
+<span class="sourceLineNo">133</span>  public void updateScanTime(long t) {<a 
name="line.133"></a>
+<span class="sourceLineNo">134</span>    serverSource.updateScanTime(t);<a 
name="line.134"></a>
+<span class="sourceLineNo">135</span>  }<a name="line.135"></a>
+<span class="sourceLineNo">136</span><a name="line.136"></a>
+<span class="sourceLineNo">137</span>  public void updateSplitTime(long t) {<a 
name="line.137"></a>
+<span class="sourceLineNo">138</span>    serverSource.updateSplitTime(t);<a 
name="line.138"></a>
+<span class="sourceLineNo">139</span>  }<a name="line.139"></a>
+<span class="sourceLineNo">140</span><a name="line.140"></a>
+<span class="sourceLineNo">141</span>  public void incrSplitRequest() {<a 
name="line.141"></a>
+<span class="sourceLineNo">142</span>    serverSource.incrSplitRequest();<a 
name="line.142"></a>
+<span class="sourceLineNo">143</span>  }<a name="line.143"></a>
+<span class="sourceLineNo">144</span><a name="line.144"></a>
+<span class="sourceLineNo">145</span>  public void incrSplitSuccess() {<a 
name="line.145"></a>
+<span class="sourceLineNo">146</span>    serverSource.incrSplitSuccess();<a 
name="line.146"></a>
+<span class="sourceLineNo">147</span>  }<a name="line.147"></a>
+<span class="sourceLineNo">148</span><a name="line.148"></a>
+<span class="sourceLineNo">149</span>  public void updateFlush(long t, long 
memstoreSize, long fileSize) {<a name="line.149"></a>
+<span class="sourceLineNo">150</span>    serverSource.updateFlushTime(t);<a 
name="line.150"></a>
+<span class="sourceLineNo">151</span>    
serverSource.updateFlushMemstoreSize(memstoreSize);<a name="line.151"></a>
+<span class="sourceLineNo">152</span>    
serverSource.updateFlushOutputSize(fileSize);<a name="line.152"></a>
+<span class="sourceLineNo">153</span>  }<a name="line.153"></a>
+<span class="sourceLineNo">154</span><a name="line.154"></a>
+<span class="sourceLineNo">155</span>  public void updateCompaction(boolean 
isMajor, long t, int inputFileCount, int outputFileCount,<a name="line.155"></a>
+<span class="sourceLineNo">156</span>      long inputBytes, long outputBytes) 
{<a name="line.156"></a>
+<span class="sourceLineNo">157</span>    
serverSource.updateCompactionTime(isMajor, t);<a name="line.157"></a>
+<span class="sourceLineNo">158</span>    
serverSource.updateCompactionInputFileCount(isMajor, inputFileCount);<a 
name="line.158"></a>
+<span class="sourceLineNo">159</span>    
serverSource.updateCompactionOutputFileCount(isMajor, outputFileCount);<a 
name="line.159"></a>
+<span class="sourceLineNo">160</span>    
serverSource.updateCompactionInputSize(isMajor, inputBytes);<a 
name="line.160"></a>
+<span class="sourceLineNo">161</span>    
serverSource.updateCompactionOutputSize(isMajor, outputBytes);<a 
name="line.161"></a>
+<span class="sourceLineNo">162</span>  }<a name="line.162"></a>
+<span class="sourceLineNo">163</span><a name="line.163"></a>
+<span class="sourceLineNo">164</span>  public void updateBulkLoad(long millis) 
{<a name="line.164"></a>
+<span class="sourceLineNo">165</span>    
this.bulkLoadTimer.updateMillis(millis);<a name="line.165"></a>
+<span class="sourceLineNo">166</span>  }<a name="line.166"></a>
+<span class="sourceLineNo">167</span>}<a name="line.167"></a>
 
 
 

Reply via email to