http://git-wip-us.apache.org/repos/asf/hbase-site/blob/193b4259/devapidocs/src-html/org/apache/hadoop/hbase/client/AsyncProcess.html
----------------------------------------------------------------------
diff --git 
a/devapidocs/src-html/org/apache/hadoop/hbase/client/AsyncProcess.html 
b/devapidocs/src-html/org/apache/hadoop/hbase/client/AsyncProcess.html
index 2c4f22d..3e8094f 100644
--- a/devapidocs/src-html/org/apache/hadoop/hbase/client/AsyncProcess.html
+++ b/devapidocs/src-html/org/apache/hadoop/hbase/client/AsyncProcess.html
@@ -73,461 +73,421 @@
 <span class="sourceLineNo">065</span> * The class manages internally the 
retries.<a name="line.65"></a>
 <span class="sourceLineNo">066</span> * &lt;/p&gt;<a name="line.66"></a>
 <span class="sourceLineNo">067</span> * &lt;p&gt;<a name="line.67"></a>
-<span class="sourceLineNo">068</span> * The class can be constructed in 
regular mode, or "global error" mode. In global error mode,<a 
name="line.68"></a>
-<span class="sourceLineNo">069</span> * AP tracks errors across all calls 
(each "future" also has global view of all errors). That<a name="line.69"></a>
-<span class="sourceLineNo">070</span> * mode is necessary for backward compat 
with HTable behavior, where multiple submissions are<a name="line.70"></a>
-<span class="sourceLineNo">071</span> * made and the errors can propagate 
using any put/flush call, from previous calls.<a name="line.71"></a>
-<span class="sourceLineNo">072</span> * In "regular" mode, the errors are 
tracked inside the Future object that is returned.<a name="line.72"></a>
-<span class="sourceLineNo">073</span> * The results are always tracked inside 
the Future object and can be retrieved when the call<a name="line.73"></a>
-<span class="sourceLineNo">074</span> * has finished. Partial results can also 
be retrieved if some part of multi-request failed.<a name="line.74"></a>
-<span class="sourceLineNo">075</span> * &lt;/p&gt;<a name="line.75"></a>
-<span class="sourceLineNo">076</span> * &lt;p&gt;<a name="line.76"></a>
-<span class="sourceLineNo">077</span> * This class is thread safe in regular 
mode; in global error code, submitting operations and<a name="line.77"></a>
-<span class="sourceLineNo">078</span> * retrieving errors from different 
threads may be not thread safe.<a name="line.78"></a>
-<span class="sourceLineNo">079</span> * Internally, the class is thread safe 
enough to manage simultaneously new submission and results<a name="line.79"></a>
-<span class="sourceLineNo">080</span> * arising from older operations.<a 
name="line.80"></a>
-<span class="sourceLineNo">081</span> * &lt;/p&gt;<a name="line.81"></a>
-<span class="sourceLineNo">082</span> * &lt;p&gt;<a name="line.82"></a>
-<span class="sourceLineNo">083</span> * Internally, this class works with 
{@link Row}, this mean it could be theoretically used for<a name="line.83"></a>
-<span class="sourceLineNo">084</span> * gets as well.<a name="line.84"></a>
-<span class="sourceLineNo">085</span> * &lt;/p&gt;<a name="line.85"></a>
-<span class="sourceLineNo">086</span> */<a name="line.86"></a>
-<span class="sourceLineNo">087</span>@InterfaceAudience.Private<a 
name="line.87"></a>
-<span class="sourceLineNo">088</span>@InterfaceStability.Evolving<a 
name="line.88"></a>
-<span class="sourceLineNo">089</span>class AsyncProcess {<a name="line.89"></a>
-<span class="sourceLineNo">090</span>  private static final Logger LOG = 
LoggerFactory.getLogger(AsyncProcess.class);<a name="line.90"></a>
-<span class="sourceLineNo">091</span>  private static final AtomicLong COUNTER 
= new AtomicLong();<a name="line.91"></a>
-<span class="sourceLineNo">092</span><a name="line.92"></a>
-<span class="sourceLineNo">093</span>  public static final String 
PRIMARY_CALL_TIMEOUT_KEY = "hbase.client.primaryCallTimeout.multiget";<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>   * Configure the number of failures 
after which the client will start logging. A few failures<a name="line.96"></a>
-<span class="sourceLineNo">097</span>   * is fine: region moved, then is not 
opened, then is overloaded. We try to have an acceptable<a name="line.97"></a>
-<span class="sourceLineNo">098</span>   * heuristic for the number of errors 
we don't log. 5 was chosen because we wait for 1s at<a name="line.98"></a>
-<span class="sourceLineNo">099</span>   * this stage.<a name="line.99"></a>
-<span class="sourceLineNo">100</span>   */<a name="line.100"></a>
-<span class="sourceLineNo">101</span>  public static final String 
START_LOG_ERRORS_AFTER_COUNT_KEY =<a name="line.101"></a>
-<span class="sourceLineNo">102</span>      
"hbase.client.start.log.errors.counter";<a name="line.102"></a>
-<span class="sourceLineNo">103</span>  public static final int 
DEFAULT_START_LOG_ERRORS_AFTER_COUNT = 5;<a name="line.103"></a>
+<span class="sourceLineNo">068</span> * The errors are tracked inside the 
Future object that is returned.<a name="line.68"></a>
+<span class="sourceLineNo">069</span> * The results are always tracked inside 
the Future object and can be retrieved when the call<a name="line.69"></a>
+<span class="sourceLineNo">070</span> * has finished. Partial results can also 
be retrieved if some part of multi-request failed.<a name="line.70"></a>
+<span class="sourceLineNo">071</span> * &lt;/p&gt;<a name="line.71"></a>
+<span class="sourceLineNo">072</span> * &lt;p&gt;<a name="line.72"></a>
+<span class="sourceLineNo">073</span> * This class is thread safe.<a 
name="line.73"></a>
+<span class="sourceLineNo">074</span> * Internally, the class is thread safe 
enough to manage simultaneously new submission and results<a name="line.74"></a>
+<span class="sourceLineNo">075</span> * arising from older operations.<a 
name="line.75"></a>
+<span class="sourceLineNo">076</span> * &lt;/p&gt;<a name="line.76"></a>
+<span class="sourceLineNo">077</span> * &lt;p&gt;<a name="line.77"></a>
+<span class="sourceLineNo">078</span> * Internally, this class works with 
{@link Row}, this mean it could be theoretically used for<a name="line.78"></a>
+<span class="sourceLineNo">079</span> * gets as well.<a name="line.79"></a>
+<span class="sourceLineNo">080</span> * &lt;/p&gt;<a name="line.80"></a>
+<span class="sourceLineNo">081</span> */<a name="line.81"></a>
+<span class="sourceLineNo">082</span>@InterfaceAudience.Private<a 
name="line.82"></a>
+<span class="sourceLineNo">083</span>@InterfaceStability.Evolving<a 
name="line.83"></a>
+<span class="sourceLineNo">084</span>class AsyncProcess {<a name="line.84"></a>
+<span class="sourceLineNo">085</span>  private static final Logger LOG = 
LoggerFactory.getLogger(AsyncProcess.class);<a name="line.85"></a>
+<span class="sourceLineNo">086</span>  private static final AtomicLong COUNTER 
= new AtomicLong();<a name="line.86"></a>
+<span class="sourceLineNo">087</span><a name="line.87"></a>
+<span class="sourceLineNo">088</span>  public static final String 
PRIMARY_CALL_TIMEOUT_KEY = "hbase.client.primaryCallTimeout.multiget";<a 
name="line.88"></a>
+<span class="sourceLineNo">089</span><a name="line.89"></a>
+<span class="sourceLineNo">090</span>  /**<a name="line.90"></a>
+<span class="sourceLineNo">091</span>   * Configure the number of failures 
after which the client will start logging. A few failures<a name="line.91"></a>
+<span class="sourceLineNo">092</span>   * is fine: region moved, then is not 
opened, then is overloaded. We try to have an acceptable<a name="line.92"></a>
+<span class="sourceLineNo">093</span>   * heuristic for the number of errors 
we don't log. 5 was chosen because we wait for 1s at<a name="line.93"></a>
+<span class="sourceLineNo">094</span>   * this stage.<a name="line.94"></a>
+<span class="sourceLineNo">095</span>   */<a name="line.95"></a>
+<span class="sourceLineNo">096</span>  public static final String 
START_LOG_ERRORS_AFTER_COUNT_KEY =<a name="line.96"></a>
+<span class="sourceLineNo">097</span>      
"hbase.client.start.log.errors.counter";<a name="line.97"></a>
+<span class="sourceLineNo">098</span>  public static final int 
DEFAULT_START_LOG_ERRORS_AFTER_COUNT = 5;<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>   * Configuration to decide whether to 
log details for batch error<a name="line.101"></a>
+<span class="sourceLineNo">102</span>   */<a name="line.102"></a>
+<span class="sourceLineNo">103</span>  public static final String 
LOG_DETAILS_FOR_BATCH_ERROR = "hbase.client.log.batcherrors.details";<a 
name="line.103"></a>
 <span class="sourceLineNo">104</span><a name="line.104"></a>
 <span class="sourceLineNo">105</span>  /**<a name="line.105"></a>
-<span class="sourceLineNo">106</span>   * Configuration to decide whether to 
log details for batch error<a name="line.106"></a>
+<span class="sourceLineNo">106</span>   * Return value from a submit that 
didn't contain any requests.<a name="line.106"></a>
 <span class="sourceLineNo">107</span>   */<a name="line.107"></a>
-<span class="sourceLineNo">108</span>  public static final String 
LOG_DETAILS_FOR_BATCH_ERROR = "hbase.client.log.batcherrors.details";<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>   * Return value from a submit that 
didn't contain any requests.<a name="line.111"></a>
-<span class="sourceLineNo">112</span>   */<a name="line.112"></a>
-<span class="sourceLineNo">113</span>  private static final AsyncRequestFuture 
NO_REQS_RESULT = new AsyncRequestFuture() {<a name="line.113"></a>
-<span class="sourceLineNo">114</span>    final Object[] result = new 
Object[0];<a name="line.114"></a>
+<span class="sourceLineNo">108</span>  private static final AsyncRequestFuture 
NO_REQS_RESULT = new AsyncRequestFuture() {<a name="line.108"></a>
+<span class="sourceLineNo">109</span>    final Object[] result = new 
Object[0];<a name="line.109"></a>
+<span class="sourceLineNo">110</span><a name="line.110"></a>
+<span class="sourceLineNo">111</span>    @Override<a name="line.111"></a>
+<span class="sourceLineNo">112</span>    public boolean hasError() {<a 
name="line.112"></a>
+<span class="sourceLineNo">113</span>      return false;<a name="line.113"></a>
+<span class="sourceLineNo">114</span>    }<a name="line.114"></a>
 <span class="sourceLineNo">115</span><a name="line.115"></a>
 <span class="sourceLineNo">116</span>    @Override<a name="line.116"></a>
-<span class="sourceLineNo">117</span>    public boolean hasError() {<a 
name="line.117"></a>
-<span class="sourceLineNo">118</span>      return false;<a name="line.118"></a>
+<span class="sourceLineNo">117</span>    public 
RetriesExhaustedWithDetailsException getErrors() {<a name="line.117"></a>
+<span class="sourceLineNo">118</span>      return null;<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>    @Override<a name="line.121"></a>
-<span class="sourceLineNo">122</span>    public 
RetriesExhaustedWithDetailsException getErrors() {<a name="line.122"></a>
+<span class="sourceLineNo">122</span>    public List&lt;? extends Row&gt; 
getFailedOperations() {<a name="line.122"></a>
 <span class="sourceLineNo">123</span>      return null;<a name="line.123"></a>
 <span class="sourceLineNo">124</span>    }<a name="line.124"></a>
 <span class="sourceLineNo">125</span><a name="line.125"></a>
 <span class="sourceLineNo">126</span>    @Override<a name="line.126"></a>
-<span class="sourceLineNo">127</span>    public List&lt;? extends Row&gt; 
getFailedOperations() {<a name="line.127"></a>
-<span class="sourceLineNo">128</span>      return null;<a name="line.128"></a>
+<span class="sourceLineNo">127</span>    public Object[] getResults() {<a 
name="line.127"></a>
+<span class="sourceLineNo">128</span>      return result;<a 
name="line.128"></a>
 <span class="sourceLineNo">129</span>    }<a name="line.129"></a>
 <span class="sourceLineNo">130</span><a name="line.130"></a>
 <span class="sourceLineNo">131</span>    @Override<a name="line.131"></a>
-<span class="sourceLineNo">132</span>    public Object[] getResults() {<a 
name="line.132"></a>
-<span class="sourceLineNo">133</span>      return result;<a 
name="line.133"></a>
-<span class="sourceLineNo">134</span>    }<a name="line.134"></a>
+<span class="sourceLineNo">132</span>    public void waitUntilDone() throws 
InterruptedIOException {<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><a name="line.135"></a>
-<span class="sourceLineNo">136</span>    @Override<a name="line.136"></a>
-<span class="sourceLineNo">137</span>    public void waitUntilDone() throws 
InterruptedIOException {<a name="line.137"></a>
-<span class="sourceLineNo">138</span>    }<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>  // TODO: many of the fields should be 
made private<a name="line.141"></a>
-<span class="sourceLineNo">142</span>  final long id;<a name="line.142"></a>
-<span class="sourceLineNo">143</span><a name="line.143"></a>
-<span class="sourceLineNo">144</span>  final ClusterConnection connection;<a 
name="line.144"></a>
-<span class="sourceLineNo">145</span>  private final RpcRetryingCallerFactory 
rpcCallerFactory;<a name="line.145"></a>
-<span class="sourceLineNo">146</span>  final RpcControllerFactory 
rpcFactory;<a name="line.146"></a>
-<span class="sourceLineNo">147</span>  final BatchErrors globalErrors;<a 
name="line.147"></a>
-<span class="sourceLineNo">148</span><a name="line.148"></a>
-<span class="sourceLineNo">149</span>  // Start configuration settings.<a 
name="line.149"></a>
-<span class="sourceLineNo">150</span>  final int startLogErrorsCnt;<a 
name="line.150"></a>
-<span class="sourceLineNo">151</span><a name="line.151"></a>
-<span class="sourceLineNo">152</span>  final long pause;<a name="line.152"></a>
-<span class="sourceLineNo">153</span>  final long pauseForCQTBE;// pause for 
CallQueueTooBigException, if specified<a name="line.153"></a>
-<span class="sourceLineNo">154</span>  final int numTries;<a 
name="line.154"></a>
-<span class="sourceLineNo">155</span>  @VisibleForTesting<a 
name="line.155"></a>
-<span class="sourceLineNo">156</span>  long serverTrackerTimeout;<a 
name="line.156"></a>
-<span class="sourceLineNo">157</span>  final long 
primaryCallTimeoutMicroseconds;<a name="line.157"></a>
-<span class="sourceLineNo">158</span>  /** Whether to log details for batch 
errors */<a name="line.158"></a>
-<span class="sourceLineNo">159</span>  final boolean logBatchErrorDetails;<a 
name="line.159"></a>
-<span class="sourceLineNo">160</span>  // End configuration settings.<a 
name="line.160"></a>
-<span class="sourceLineNo">161</span><a name="line.161"></a>
-<span class="sourceLineNo">162</span>  /**<a name="line.162"></a>
-<span class="sourceLineNo">163</span>   * The traffic control for requests.<a 
name="line.163"></a>
-<span class="sourceLineNo">164</span>   */<a name="line.164"></a>
-<span class="sourceLineNo">165</span>  @VisibleForTesting<a 
name="line.165"></a>
-<span class="sourceLineNo">166</span>  final RequestController 
requestController;<a name="line.166"></a>
-<span class="sourceLineNo">167</span>  public static final String 
LOG_DETAILS_PERIOD = "hbase.client.log.detail.period.ms";<a name="line.167"></a>
-<span class="sourceLineNo">168</span>  private static final int 
DEFAULT_LOG_DETAILS_PERIOD = 10000;<a name="line.168"></a>
-<span class="sourceLineNo">169</span>  private final int periodToLog;<a 
name="line.169"></a>
-<span class="sourceLineNo">170</span>  AsyncProcess(ClusterConnection hc, 
Configuration conf,<a name="line.170"></a>
-<span class="sourceLineNo">171</span>      RpcRetryingCallerFactory rpcCaller, 
boolean useGlobalErrors,<a name="line.171"></a>
-<span class="sourceLineNo">172</span>      RpcControllerFactory rpcFactory) 
{<a name="line.172"></a>
-<span class="sourceLineNo">173</span>    if (hc == null) {<a 
name="line.173"></a>
-<span class="sourceLineNo">174</span>      throw new 
IllegalArgumentException("ClusterConnection cannot be null.");<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>    this.connection = hc;<a 
name="line.177"></a>
-<span class="sourceLineNo">178</span>    this.globalErrors = useGlobalErrors ? 
new BatchErrors() : null;<a name="line.178"></a>
-<span class="sourceLineNo">179</span><a name="line.179"></a>
-<span class="sourceLineNo">180</span>    this.id = 
COUNTER.incrementAndGet();<a name="line.180"></a>
-<span class="sourceLineNo">181</span><a name="line.181"></a>
-<span class="sourceLineNo">182</span>    this.pause = 
conf.getLong(HConstants.HBASE_CLIENT_PAUSE,<a name="line.182"></a>
-<span class="sourceLineNo">183</span>        
HConstants.DEFAULT_HBASE_CLIENT_PAUSE);<a name="line.183"></a>
-<span class="sourceLineNo">184</span>    long configuredPauseForCQTBE = 
conf.getLong(HConstants.HBASE_CLIENT_PAUSE_FOR_CQTBE, pause);<a 
name="line.184"></a>
-<span class="sourceLineNo">185</span>    if (configuredPauseForCQTBE &lt; 
pause) {<a name="line.185"></a>
-<span class="sourceLineNo">186</span>      LOG.warn("The " + 
HConstants.HBASE_CLIENT_PAUSE_FOR_CQTBE + " setting: "<a name="line.186"></a>
-<span class="sourceLineNo">187</span>          + configuredPauseForCQTBE + " 
is smaller than " + HConstants.HBASE_CLIENT_PAUSE<a name="line.187"></a>
-<span class="sourceLineNo">188</span>          + ", will use " + pause + " 
instead.");<a name="line.188"></a>
-<span class="sourceLineNo">189</span>      this.pauseForCQTBE = pause;<a 
name="line.189"></a>
-<span class="sourceLineNo">190</span>    } else {<a name="line.190"></a>
-<span class="sourceLineNo">191</span>      this.pauseForCQTBE = 
configuredPauseForCQTBE;<a name="line.191"></a>
-<span class="sourceLineNo">192</span>    }<a name="line.192"></a>
-<span class="sourceLineNo">193</span>    // how many times we could try in 
total, one more than retry number<a name="line.193"></a>
-<span class="sourceLineNo">194</span>    this.numTries = 
conf.getInt(HConstants.HBASE_CLIENT_RETRIES_NUMBER,<a name="line.194"></a>
-<span class="sourceLineNo">195</span>        
HConstants.DEFAULT_HBASE_CLIENT_RETRIES_NUMBER) + 1;<a name="line.195"></a>
-<span class="sourceLineNo">196</span>    this.primaryCallTimeoutMicroseconds = 
conf.getInt(PRIMARY_CALL_TIMEOUT_KEY, 10000);<a name="line.196"></a>
-<span class="sourceLineNo">197</span>    this.startLogErrorsCnt =<a 
name="line.197"></a>
-<span class="sourceLineNo">198</span>        
conf.getInt(START_LOG_ERRORS_AFTER_COUNT_KEY, 
DEFAULT_START_LOG_ERRORS_AFTER_COUNT);<a name="line.198"></a>
-<span class="sourceLineNo">199</span>    this.periodToLog = 
conf.getInt(LOG_DETAILS_PERIOD, DEFAULT_LOG_DETAILS_PERIOD);<a 
name="line.199"></a>
-<span class="sourceLineNo">200</span>    // Server tracker allows us to do 
faster, and yet useful (hopefully), retries.<a name="line.200"></a>
-<span class="sourceLineNo">201</span>    // However, if we are too useful, we 
might fail very quickly due to retry count limit.<a name="line.201"></a>
-<span class="sourceLineNo">202</span>    // To avoid this, we are going to 
cheat for now (see HBASE-7659), and calculate maximum<a name="line.202"></a>
-<span class="sourceLineNo">203</span>    // retry time if normal retries were 
used. Then we will retry until this time runs out.<a name="line.203"></a>
-<span class="sourceLineNo">204</span>    // If we keep hitting one server, the 
net effect will be the incremental backoff, and<a name="line.204"></a>
-<span class="sourceLineNo">205</span>    // essentially the same number of 
retries as planned. If we have to do faster retries,<a name="line.205"></a>
-<span class="sourceLineNo">206</span>    // we will do more retries in 
aggregate, but the user will be none the wiser.<a name="line.206"></a>
-<span class="sourceLineNo">207</span>    this.serverTrackerTimeout = 0L;<a 
name="line.207"></a>
-<span class="sourceLineNo">208</span>    for (int i = 0; i &lt; this.numTries; 
++i) {<a name="line.208"></a>
-<span class="sourceLineNo">209</span>      serverTrackerTimeout = 
serverTrackerTimeout + ConnectionUtils.getPauseTime(this.pause, i);<a 
name="line.209"></a>
-<span class="sourceLineNo">210</span>    }<a name="line.210"></a>
-<span class="sourceLineNo">211</span><a name="line.211"></a>
-<span class="sourceLineNo">212</span>    this.rpcCallerFactory = rpcCaller;<a 
name="line.212"></a>
-<span class="sourceLineNo">213</span>    this.rpcFactory = rpcFactory;<a 
name="line.213"></a>
-<span class="sourceLineNo">214</span>    this.logBatchErrorDetails = 
conf.getBoolean(LOG_DETAILS_FOR_BATCH_ERROR, false);<a name="line.214"></a>
-<span class="sourceLineNo">215</span><a name="line.215"></a>
-<span class="sourceLineNo">216</span>    this.requestController = 
RequestControllerFactory.create(conf);<a name="line.216"></a>
-<span class="sourceLineNo">217</span>  }<a name="line.217"></a>
-<span class="sourceLineNo">218</span><a name="line.218"></a>
-<span class="sourceLineNo">219</span>  /**<a name="line.219"></a>
-<span class="sourceLineNo">220</span>   * The submitted task may be not 
accomplished at all if there are too many running tasks or<a 
name="line.220"></a>
-<span class="sourceLineNo">221</span>   * other limits.<a name="line.221"></a>
-<span class="sourceLineNo">222</span>   * @param &lt;CResult&gt; The class to 
cast the result<a name="line.222"></a>
-<span class="sourceLineNo">223</span>   * @param task The setting and data<a 
name="line.223"></a>
-<span class="sourceLineNo">224</span>   * @return AsyncRequestFuture<a 
name="line.224"></a>
-<span class="sourceLineNo">225</span>   */<a name="line.225"></a>
-<span class="sourceLineNo">226</span>  public &lt;CResult&gt; 
AsyncRequestFuture submit(AsyncProcessTask&lt;CResult&gt; task) throws 
InterruptedIOException {<a name="line.226"></a>
-<span class="sourceLineNo">227</span>    AsyncRequestFuture reqFuture = 
checkTask(task);<a name="line.227"></a>
-<span class="sourceLineNo">228</span>    if (reqFuture != null) {<a 
name="line.228"></a>
-<span class="sourceLineNo">229</span>      return reqFuture;<a 
name="line.229"></a>
-<span class="sourceLineNo">230</span>    }<a name="line.230"></a>
-<span class="sourceLineNo">231</span>    SubmittedRows submittedRows = 
task.getSubmittedRows() == null ? SubmittedRows.ALL : 
task.getSubmittedRows();<a name="line.231"></a>
-<span class="sourceLineNo">232</span>    switch (submittedRows) {<a 
name="line.232"></a>
-<span class="sourceLineNo">233</span>      case ALL:<a name="line.233"></a>
-<span class="sourceLineNo">234</span>        return submitAll(task);<a 
name="line.234"></a>
-<span class="sourceLineNo">235</span>      case AT_LEAST_ONE:<a 
name="line.235"></a>
-<span class="sourceLineNo">236</span>        return submit(task, true);<a 
name="line.236"></a>
-<span class="sourceLineNo">237</span>      default:<a name="line.237"></a>
-<span class="sourceLineNo">238</span>        return submit(task, false);<a 
name="line.238"></a>
-<span class="sourceLineNo">239</span>    }<a name="line.239"></a>
-<span class="sourceLineNo">240</span>  }<a name="line.240"></a>
-<span class="sourceLineNo">241</span><a name="line.241"></a>
-<span class="sourceLineNo">242</span>  /**<a name="line.242"></a>
-<span class="sourceLineNo">243</span>   * Extract from the rows list what we 
can submit. The rows we can not submit are kept in the<a name="line.243"></a>
-<span class="sourceLineNo">244</span>   * list. Does not send requests to 
replicas (not currently used for anything other<a name="line.244"></a>
-<span class="sourceLineNo">245</span>   * than streaming puts anyway).<a 
name="line.245"></a>
-<span class="sourceLineNo">246</span>   *<a name="line.246"></a>
-<span class="sourceLineNo">247</span>   * @param task The setting and data<a 
name="line.247"></a>
-<span class="sourceLineNo">248</span>   * @param atLeastOne true if we should 
submit at least a subset.<a name="line.248"></a>
-<span class="sourceLineNo">249</span>   */<a name="line.249"></a>
-<span class="sourceLineNo">250</span>  private &lt;CResult&gt; 
AsyncRequestFuture submit(AsyncProcessTask&lt;CResult&gt; task,<a 
name="line.250"></a>
-<span class="sourceLineNo">251</span>    boolean atLeastOne) throws 
InterruptedIOException {<a name="line.251"></a>
-<span class="sourceLineNo">252</span>    TableName tableName = 
task.getTableName();<a name="line.252"></a>
-<span class="sourceLineNo">253</span>    RowAccess&lt;? extends Row&gt; rows = 
task.getRowAccess();<a name="line.253"></a>
-<span class="sourceLineNo">254</span>    Map&lt;ServerName, MultiAction&gt; 
actionsByServer = new HashMap&lt;&gt;();<a name="line.254"></a>
-<span class="sourceLineNo">255</span>    List&lt;Action&gt; retainedActions = 
new ArrayList&lt;&gt;(rows.size());<a name="line.255"></a>
-<span class="sourceLineNo">256</span><a name="line.256"></a>
-<span class="sourceLineNo">257</span>    NonceGenerator ng = 
this.connection.getNonceGenerator();<a name="line.257"></a>
-<span class="sourceLineNo">258</span>    long nonceGroup = ng.getNonceGroup(); 
// Currently, nonce group is per entire client.<a name="line.258"></a>
-<span class="sourceLineNo">259</span><a name="line.259"></a>
-<span class="sourceLineNo">260</span>    // Location errors that happen before 
we decide what requests to take.<a name="line.260"></a>
-<span class="sourceLineNo">261</span>    List&lt;Exception&gt; locationErrors 
= null;<a name="line.261"></a>
-<span class="sourceLineNo">262</span>    List&lt;Integer&gt; locationErrorRows 
= null;<a name="line.262"></a>
-<span class="sourceLineNo">263</span>    RequestController.Checker checker = 
requestController.newChecker();<a name="line.263"></a>
-<span class="sourceLineNo">264</span>    boolean firstIter = true;<a 
name="line.264"></a>
-<span class="sourceLineNo">265</span>    do {<a name="line.265"></a>
-<span class="sourceLineNo">266</span>      // Wait until there is at least one 
slot for a new task.<a name="line.266"></a>
-<span class="sourceLineNo">267</span>      
requestController.waitForFreeSlot(id, periodToLog, getLogger(tableName, -1));<a 
name="line.267"></a>
-<span class="sourceLineNo">268</span>      int posInList = -1;<a 
name="line.268"></a>
-<span class="sourceLineNo">269</span>      if (!firstIter) {<a 
name="line.269"></a>
-<span class="sourceLineNo">270</span>        checker.reset();<a 
name="line.270"></a>
-<span class="sourceLineNo">271</span>      }<a name="line.271"></a>
-<span class="sourceLineNo">272</span>      Iterator&lt;? extends Row&gt; it = 
rows.iterator();<a name="line.272"></a>
-<span class="sourceLineNo">273</span>      while (it.hasNext()) {<a 
name="line.273"></a>
-<span class="sourceLineNo">274</span>        Row r = it.next();<a 
name="line.274"></a>
-<span class="sourceLineNo">275</span>        HRegionLocation loc;<a 
name="line.275"></a>
-<span class="sourceLineNo">276</span>        try {<a name="line.276"></a>
-<span class="sourceLineNo">277</span>          if (r == null) {<a 
name="line.277"></a>
-<span class="sourceLineNo">278</span>            throw new 
IllegalArgumentException("#" + id + ", row cannot be null");<a 
name="line.278"></a>
-<span class="sourceLineNo">279</span>          }<a name="line.279"></a>
-<span class="sourceLineNo">280</span>          // Make sure we get 0-s 
replica.<a name="line.280"></a>
-<span class="sourceLineNo">281</span>          RegionLocations locs = 
connection.locateRegion(<a name="line.281"></a>
-<span class="sourceLineNo">282</span>              tableName, r.getRow(), 
true, true, RegionReplicaUtil.DEFAULT_REPLICA_ID);<a name="line.282"></a>
-<span class="sourceLineNo">283</span>          if (locs == null || 
locs.isEmpty() || locs.getDefaultRegionLocation() == null) {<a 
name="line.283"></a>
-<span class="sourceLineNo">284</span>            throw new IOException("#" + 
id + ", no location found, aborting submit for"<a name="line.284"></a>
-<span class="sourceLineNo">285</span>                + " tableName=" + 
tableName + " rowkey=" + Bytes.toStringBinary(r.getRow()));<a 
name="line.285"></a>
-<span class="sourceLineNo">286</span>          }<a name="line.286"></a>
-<span class="sourceLineNo">287</span>          loc = 
locs.getDefaultRegionLocation();<a name="line.287"></a>
-<span class="sourceLineNo">288</span>        } catch (IOException ex) {<a 
name="line.288"></a>
-<span class="sourceLineNo">289</span>          locationErrors = new 
ArrayList&lt;&gt;(1);<a name="line.289"></a>
-<span class="sourceLineNo">290</span>          locationErrorRows = new 
ArrayList&lt;&gt;(1);<a name="line.290"></a>
-<span class="sourceLineNo">291</span>          LOG.error("Failed to get region 
location ", ex);<a name="line.291"></a>
-<span class="sourceLineNo">292</span>          // This action failed before 
creating ars. Retain it, but do not add to submit list.<a name="line.292"></a>
-<span class="sourceLineNo">293</span>          // We will then add it to ars 
in an already-failed state.<a name="line.293"></a>
-<span class="sourceLineNo">294</span><a name="line.294"></a>
-<span class="sourceLineNo">295</span>          int priority = 
HConstants.NORMAL_QOS;<a name="line.295"></a>
-<span class="sourceLineNo">296</span>          if (r instanceof Mutation) {<a 
name="line.296"></a>
-<span class="sourceLineNo">297</span>            priority = ((Mutation) 
r).getPriority();<a name="line.297"></a>
-<span class="sourceLineNo">298</span>          }<a name="line.298"></a>
-<span class="sourceLineNo">299</span>          retainedActions.add(new 
Action(r, ++posInList, priority));<a name="line.299"></a>
-<span class="sourceLineNo">300</span>          locationErrors.add(ex);<a 
name="line.300"></a>
-<span class="sourceLineNo">301</span>          
locationErrorRows.add(posInList);<a name="line.301"></a>
-<span class="sourceLineNo">302</span>          it.remove();<a 
name="line.302"></a>
-<span class="sourceLineNo">303</span>          break; // Backward compat: we 
stop considering actions on location error.<a name="line.303"></a>
-<span class="sourceLineNo">304</span>        }<a name="line.304"></a>
-<span class="sourceLineNo">305</span>        ReturnCode code = 
checker.canTakeRow(loc, r);<a name="line.305"></a>
-<span class="sourceLineNo">306</span>        if (code == ReturnCode.END) {<a 
name="line.306"></a>
-<span class="sourceLineNo">307</span>          break;<a name="line.307"></a>
-<span class="sourceLineNo">308</span>        }<a name="line.308"></a>
-<span class="sourceLineNo">309</span>        if (code == ReturnCode.INCLUDE) 
{<a name="line.309"></a>
-<span class="sourceLineNo">310</span>          int priority = 
HConstants.NORMAL_QOS;<a name="line.310"></a>
-<span class="sourceLineNo">311</span>          if (r instanceof Mutation) {<a 
name="line.311"></a>
-<span class="sourceLineNo">312</span>            priority = ((Mutation) 
r).getPriority();<a name="line.312"></a>
-<span class="sourceLineNo">313</span>          }<a name="line.313"></a>
-<span class="sourceLineNo">314</span>          Action action = new Action(r, 
++posInList, priority);<a name="line.314"></a>
-<span class="sourceLineNo">315</span>          setNonce(ng, r, action);<a 
name="line.315"></a>
-<span class="sourceLineNo">316</span>          retainedActions.add(action);<a 
name="line.316"></a>
-<span class="sourceLineNo">317</span>          // TODO: replica-get is not 
supported on this path<a name="line.317"></a>
-<span class="sourceLineNo">318</span>          byte[] regionName = 
loc.getRegionInfo().getRegionName();<a name="line.318"></a>
-<span class="sourceLineNo">319</span>          addAction(loc.getServerName(), 
regionName, action, actionsByServer, nonceGroup);<a name="line.319"></a>
-<span class="sourceLineNo">320</span>          it.remove();<a 
name="line.320"></a>
-<span class="sourceLineNo">321</span>        }<a name="line.321"></a>
-<span class="sourceLineNo">322</span>      }<a name="line.322"></a>
-<span class="sourceLineNo">323</span>      firstIter = false;<a 
name="line.323"></a>
-<span class="sourceLineNo">324</span>    } while (retainedActions.isEmpty() 
&amp;&amp; atLeastOne &amp;&amp; (locationErrors == null));<a 
name="line.324"></a>
-<span class="sourceLineNo">325</span><a name="line.325"></a>
-<span class="sourceLineNo">326</span>    if (retainedActions.isEmpty()) return 
NO_REQS_RESULT;<a name="line.326"></a>
-<span class="sourceLineNo">327</span><a name="line.327"></a>
-<span class="sourceLineNo">328</span>    return submitMultiActions(task, 
retainedActions, nonceGroup,<a name="line.328"></a>
-<span class="sourceLineNo">329</span>        locationErrors, 
locationErrorRows, actionsByServer);<a name="line.329"></a>
-<span class="sourceLineNo">330</span>  }<a name="line.330"></a>
-<span class="sourceLineNo">331</span><a name="line.331"></a>
-<span class="sourceLineNo">332</span>  &lt;CResult&gt; AsyncRequestFuture 
submitMultiActions(AsyncProcessTask task,<a name="line.332"></a>
-<span class="sourceLineNo">333</span>      List&lt;Action&gt; retainedActions, 
long nonceGroup, List&lt;Exception&gt; locationErrors,<a name="line.333"></a>
-<span class="sourceLineNo">334</span>      List&lt;Integer&gt; 
locationErrorRows, Map&lt;ServerName, MultiAction&gt; actionsByServer) {<a 
name="line.334"></a>
-<span class="sourceLineNo">335</span>    AsyncRequestFutureImpl&lt;CResult&gt; 
ars = createAsyncRequestFuture(task, retainedActions, nonceGroup);<a 
name="line.335"></a>
-<span class="sourceLineNo">336</span>    // Add location errors if any<a 
name="line.336"></a>
-<span class="sourceLineNo">337</span>    if (locationErrors != null) {<a 
name="line.337"></a>
-<span class="sourceLineNo">338</span>      for (int i = 0; i &lt; 
locationErrors.size(); ++i) {<a name="line.338"></a>
-<span class="sourceLineNo">339</span>        int originalIndex = 
locationErrorRows.get(i);<a name="line.339"></a>
-<span class="sourceLineNo">340</span>        Row row = 
retainedActions.get(originalIndex).getAction();<a name="line.340"></a>
-<span class="sourceLineNo">341</span>        ars.manageError(originalIndex, 
row,<a name="line.341"></a>
-<span class="sourceLineNo">342</span>            
AsyncRequestFutureImpl.Retry.NO_LOCATION_PROBLEM, locationErrors.get(i), 
null);<a name="line.342"></a>
-<span class="sourceLineNo">343</span>      }<a name="line.343"></a>
-<span class="sourceLineNo">344</span>    }<a name="line.344"></a>
-<span class="sourceLineNo">345</span>    ars.sendMultiAction(actionsByServer, 
1, null, false);<a name="line.345"></a>
-<span class="sourceLineNo">346</span>    return ars;<a name="line.346"></a>
-<span class="sourceLineNo">347</span>  }<a name="line.347"></a>
-<span class="sourceLineNo">348</span><a name="line.348"></a>
-<span class="sourceLineNo">349</span>  /**<a name="line.349"></a>
-<span class="sourceLineNo">350</span>   * Helper that is used when grouping 
the actions per region server.<a name="line.350"></a>
-<span class="sourceLineNo">351</span>   *<a name="line.351"></a>
-<span class="sourceLineNo">352</span>   * @param server - server<a 
name="line.352"></a>
-<span class="sourceLineNo">353</span>   * @param regionName - regionName<a 
name="line.353"></a>
-<span class="sourceLineNo">354</span>   * @param action - the action to add to 
the multiaction<a name="line.354"></a>
-<span class="sourceLineNo">355</span>   * @param actionsByServer the 
multiaction per server<a name="line.355"></a>
-<span class="sourceLineNo">356</span>   * @param nonceGroup Nonce group.<a 
name="line.356"></a>
-<span class="sourceLineNo">357</span>   */<a name="line.357"></a>
-<span class="sourceLineNo">358</span>  static void addAction(ServerName 
server, byte[] regionName, Action action,<a name="line.358"></a>
-<span class="sourceLineNo">359</span>      Map&lt;ServerName, MultiAction&gt; 
actionsByServer, long nonceGroup) {<a name="line.359"></a>
-<span class="sourceLineNo">360</span>    MultiAction multiAction = 
actionsByServer.get(server);<a name="line.360"></a>
-<span class="sourceLineNo">361</span>    if (multiAction == null) {<a 
name="line.361"></a>
-<span class="sourceLineNo">362</span>      multiAction = new MultiAction();<a 
name="line.362"></a>
-<span class="sourceLineNo">363</span>      actionsByServer.put(server, 
multiAction);<a name="line.363"></a>
-<span class="sourceLineNo">364</span>    }<a name="line.364"></a>
-<span class="sourceLineNo">365</span>    if (action.hasNonce() &amp;&amp; 
!multiAction.hasNonceGroup()) {<a name="line.365"></a>
-<span class="sourceLineNo">366</span>      
multiAction.setNonceGroup(nonceGroup);<a name="line.366"></a>
-<span class="sourceLineNo">367</span>    }<a name="line.367"></a>
-<span class="sourceLineNo">368</span><a name="line.368"></a>
-<span class="sourceLineNo">369</span>    multiAction.add(regionName, 
action);<a name="line.369"></a>
-<span class="sourceLineNo">370</span>  }<a name="line.370"></a>
-<span class="sourceLineNo">371</span><a name="line.371"></a>
-<span class="sourceLineNo">372</span>  /**<a name="line.372"></a>
-<span class="sourceLineNo">373</span>   * Submit immediately the list of rows, 
whatever the server status. Kept for backward<a name="line.373"></a>
-<span class="sourceLineNo">374</span>   * compatibility: it allows to be used 
with the batch interface that return an array of objects.<a name="line.374"></a>
-<span class="sourceLineNo">375</span>   * @param task The setting and data<a 
name="line.375"></a>
-<span class="sourceLineNo">376</span>   */<a name="line.376"></a>
-<span class="sourceLineNo">377</span>  private &lt;CResult&gt; 
AsyncRequestFuture submitAll(AsyncProcessTask task) {<a name="line.377"></a>
-<span class="sourceLineNo">378</span>    RowAccess&lt;? extends Row&gt; rows = 
task.getRowAccess();<a name="line.378"></a>
-<span class="sourceLineNo">379</span>    List&lt;Action&gt; actions = new 
ArrayList&lt;&gt;(rows.size());<a name="line.379"></a>
-<span class="sourceLineNo">380</span><a name="line.380"></a>
-<span class="sourceLineNo">381</span>    // The position will be used by the 
processBatch to match the object array returned.<a name="line.381"></a>
-<span class="sourceLineNo">382</span>    int posInList = -1;<a 
name="line.382"></a>
-<span class="sourceLineNo">383</span>    NonceGenerator ng = 
this.connection.getNonceGenerator();<a name="line.383"></a>
-<span class="sourceLineNo">384</span>    int highestPriority = 
HConstants.PRIORITY_UNSET;<a name="line.384"></a>
-<span class="sourceLineNo">385</span>    for (Row r : rows) {<a 
name="line.385"></a>
-<span class="sourceLineNo">386</span>      posInList++;<a name="line.386"></a>
-<span class="sourceLineNo">387</span>      if (r instanceof Put) {<a 
name="line.387"></a>
-<span class="sourceLineNo">388</span>        Put put = (Put) r;<a 
name="line.388"></a>
-<span class="sourceLineNo">389</span>        if (put.isEmpty()) {<a 
name="line.389"></a>
-<span class="sourceLineNo">390</span>          throw new 
IllegalArgumentException("No columns to insert for #" + (posInList+1)+ " 
item");<a name="line.390"></a>
-<span class="sourceLineNo">391</span>        }<a name="line.391"></a>
-<span class="sourceLineNo">392</span>        highestPriority = 
Math.max(put.getPriority(), highestPriority);<a name="line.392"></a>
-<span class="sourceLineNo">393</span>      }<a name="line.393"></a>
-<span class="sourceLineNo">394</span>      Action action = new Action(r, 
posInList, highestPriority);<a name="line.394"></a>
-<span class="sourceLineNo">395</span>      setNonce(ng, r, action);<a 
name="line.395"></a>
-<span class="sourceLineNo">396</span>      actions.add(action);<a 
name="line.396"></a>
-<span class="sourceLineNo">397</span>    }<a name="line.397"></a>
-<span class="sourceLineNo">398</span>    AsyncRequestFutureImpl&lt;CResult&gt; 
ars = createAsyncRequestFuture(task, actions, ng.getNonceGroup());<a 
name="line.398"></a>
-<span class="sourceLineNo">399</span>    ars.groupAndSendMultiAction(actions, 
1);<a name="line.399"></a>
-<span class="sourceLineNo">400</span>    return ars;<a name="line.400"></a>
-<span class="sourceLineNo">401</span>  }<a name="line.401"></a>
-<span class="sourceLineNo">402</span><a name="line.402"></a>
-<span class="sourceLineNo">403</span>  private &lt;CResult&gt; 
AsyncRequestFuture checkTask(AsyncProcessTask&lt;CResult&gt; task) {<a 
name="line.403"></a>
-<span class="sourceLineNo">404</span>    if (task.getRowAccess() == null || 
task.getRowAccess().isEmpty()) {<a name="line.404"></a>
-<span class="sourceLineNo">405</span>      return NO_REQS_RESULT;<a 
name="line.405"></a>
-<span class="sourceLineNo">406</span>    }<a name="line.406"></a>
-<span class="sourceLineNo">407</span>    
Objects.requireNonNull(task.getPool(), "The pool can't be NULL");<a 
name="line.407"></a>
-<span class="sourceLineNo">408</span>    
checkOperationTimeout(task.getOperationTimeout());<a name="line.408"></a>
-<span class="sourceLineNo">409</span>    
checkRpcTimeout(task.getRpcTimeout());<a name="line.409"></a>
-<span class="sourceLineNo">410</span>    return null;<a name="line.410"></a>
-<span class="sourceLineNo">411</span>  }<a name="line.411"></a>
-<span class="sourceLineNo">412</span><a name="line.412"></a>
-<span class="sourceLineNo">413</span>  private void setNonce(NonceGenerator 
ng, Row r, Action action) {<a name="line.413"></a>
-<span class="sourceLineNo">414</span>    if (!(r instanceof Append) &amp;&amp; 
!(r instanceof Increment)) return;<a name="line.414"></a>
-<span class="sourceLineNo">415</span>    action.setNonce(ng.newNonce()); // 
Action handles NO_NONCE, so it's ok if ng is disabled.<a name="line.415"></a>
+<span class="sourceLineNo">136</span>  // TODO: many of the fields should be 
made private<a name="line.136"></a>
+<span class="sourceLineNo">137</span>  final long id;<a name="line.137"></a>
+<span class="sourceLineNo">138</span><a name="line.138"></a>
+<span class="sourceLineNo">139</span>  final ClusterConnection connection;<a 
name="line.139"></a>
+<span class="sourceLineNo">140</span>  private final RpcRetryingCallerFactory 
rpcCallerFactory;<a name="line.140"></a>
+<span class="sourceLineNo">141</span>  final RpcControllerFactory 
rpcFactory;<a name="line.141"></a>
+<span class="sourceLineNo">142</span><a name="line.142"></a>
+<span class="sourceLineNo">143</span>  // Start configuration settings.<a 
name="line.143"></a>
+<span class="sourceLineNo">144</span>  final int startLogErrorsCnt;<a 
name="line.144"></a>
+<span class="sourceLineNo">145</span><a name="line.145"></a>
+<span class="sourceLineNo">146</span>  final long pause;<a name="line.146"></a>
+<span class="sourceLineNo">147</span>  final long pauseForCQTBE;// pause for 
CallQueueTooBigException, if specified<a name="line.147"></a>
+<span class="sourceLineNo">148</span>  final int numTries;<a 
name="line.148"></a>
+<span class="sourceLineNo">149</span>  @VisibleForTesting<a 
name="line.149"></a>
+<span class="sourceLineNo">150</span>  long serverTrackerTimeout;<a 
name="line.150"></a>
+<span class="sourceLineNo">151</span>  final long 
primaryCallTimeoutMicroseconds;<a name="line.151"></a>
+<span class="sourceLineNo">152</span>  /** Whether to log details for batch 
errors */<a name="line.152"></a>
+<span class="sourceLineNo">153</span>  final boolean logBatchErrorDetails;<a 
name="line.153"></a>
+<span class="sourceLineNo">154</span>  // End configuration settings.<a 
name="line.154"></a>
+<span class="sourceLineNo">155</span><a name="line.155"></a>
+<span class="sourceLineNo">156</span>  /**<a name="line.156"></a>
+<span class="sourceLineNo">157</span>   * The traffic control for requests.<a 
name="line.157"></a>
+<span class="sourceLineNo">158</span>   */<a name="line.158"></a>
+<span class="sourceLineNo">159</span>  @VisibleForTesting<a 
name="line.159"></a>
+<span class="sourceLineNo">160</span>  final RequestController 
requestController;<a name="line.160"></a>
+<span class="sourceLineNo">161</span>  public static final String 
LOG_DETAILS_PERIOD = "hbase.client.log.detail.period.ms";<a name="line.161"></a>
+<span class="sourceLineNo">162</span>  private static final int 
DEFAULT_LOG_DETAILS_PERIOD = 10000;<a name="line.162"></a>
+<span class="sourceLineNo">163</span>  private final int periodToLog;<a 
name="line.163"></a>
+<span class="sourceLineNo">164</span>  AsyncProcess(ClusterConnection hc, 
Configuration conf,<a name="line.164"></a>
+<span class="sourceLineNo">165</span>      RpcRetryingCallerFactory rpcCaller, 
RpcControllerFactory rpcFactory) {<a name="line.165"></a>
+<span class="sourceLineNo">166</span>    if (hc == null) {<a 
name="line.166"></a>
+<span class="sourceLineNo">167</span>      throw new 
IllegalArgumentException("ClusterConnection cannot be null.");<a 
name="line.167"></a>
+<span class="sourceLineNo">168</span>    }<a name="line.168"></a>
+<span class="sourceLineNo">169</span><a name="line.169"></a>
+<span class="sourceLineNo">170</span>    this.connection = hc;<a 
name="line.170"></a>
+<span class="sourceLineNo">171</span><a name="line.171"></a>
+<span class="sourceLineNo">172</span>    this.id = 
COUNTER.incrementAndGet();<a name="line.172"></a>
+<span class="sourceLineNo">173</span><a name="line.173"></a>
+<span class="sourceLineNo">174</span>    this.pause = 
conf.getLong(HConstants.HBASE_CLIENT_PAUSE,<a name="line.174"></a>
+<span class="sourceLineNo">175</span>        
HConstants.DEFAULT_HBASE_CLIENT_PAUSE);<a name="line.175"></a>
+<span class="sourceLineNo">176</span>    long configuredPauseForCQTBE = 
conf.getLong(HConstants.HBASE_CLIENT_PAUSE_FOR_CQTBE, pause);<a 
name="line.176"></a>
+<span class="sourceLineNo">177</span>    if (configuredPauseForCQTBE &lt; 
pause) {<a name="line.177"></a>
+<span class="sourceLineNo">178</span>      LOG.warn("The " + 
HConstants.HBASE_CLIENT_PAUSE_FOR_CQTBE + " setting: "<a name="line.178"></a>
+<span class="sourceLineNo">179</span>          + configuredPauseForCQTBE + " 
is smaller than " + HConstants.HBASE_CLIENT_PAUSE<a name="line.179"></a>
+<span class="sourceLineNo">180</span>          + ", will use " + pause + " 
instead.");<a name="line.180"></a>
+<span class="sourceLineNo">181</span>      this.pauseForCQTBE = pause;<a 
name="line.181"></a>
+<span class="sourceLineNo">182</span>    } else {<a name="line.182"></a>
+<span class="sourceLineNo">183</span>      this.pauseForCQTBE = 
configuredPauseForCQTBE;<a name="line.183"></a>
+<span class="sourceLineNo">184</span>    }<a name="line.184"></a>
+<span class="sourceLineNo">185</span>    // how many times we could try in 
total, one more than retry number<a name="line.185"></a>
+<span class="sourceLineNo">186</span>    this.numTries = 
conf.getInt(HConstants.HBASE_CLIENT_RETRIES_NUMBER,<a name="line.186"></a>
+<span class="sourceLineNo">187</span>        
HConstants.DEFAULT_HBASE_CLIENT_RETRIES_NUMBER) + 1;<a name="line.187"></a>
+<span class="sourceLineNo">188</span>    this.primaryCallTimeoutMicroseconds = 
conf.getInt(PRIMARY_CALL_TIMEOUT_KEY, 10000);<a name="line.188"></a>
+<span class="sourceLineNo">189</span>    this.startLogErrorsCnt =<a 
name="line.189"></a>
+<span class="sourceLineNo">190</span>        
conf.getInt(START_LOG_ERRORS_AFTER_COUNT_KEY, 
DEFAULT_START_LOG_ERRORS_AFTER_COUNT);<a name="line.190"></a>
+<span class="sourceLineNo">191</span>    this.periodToLog = 
conf.getInt(LOG_DETAILS_PERIOD, DEFAULT_LOG_DETAILS_PERIOD);<a 
name="line.191"></a>
+<span class="sourceLineNo">192</span>    // Server tracker allows us to do 
faster, and yet useful (hopefully), retries.<a name="line.192"></a>
+<span class="sourceLineNo">193</span>    // However, if we are too useful, we 
might fail very quickly due to retry count limit.<a name="line.193"></a>
+<span class="sourceLineNo">194</span>    // To avoid this, we are going to 
cheat for now (see HBASE-7659), and calculate maximum<a name="line.194"></a>
+<span class="sourceLineNo">195</span>    // retry time if normal retries were 
used. Then we will retry until this time runs out.<a name="line.195"></a>
+<span class="sourceLineNo">196</span>    // If we keep hitting one server, the 
net effect will be the incremental backoff, and<a name="line.196"></a>
+<span class="sourceLineNo">197</span>    // essentially the same number of 
retries as planned. If we have to do faster retries,<a name="line.197"></a>
+<span class="sourceLineNo">198</span>    // we will do more retries in 
aggregate, but the user will be none the wiser.<a name="line.198"></a>
+<span class="sourceLineNo">199</span>    this.serverTrackerTimeout = 0L;<a 
name="line.199"></a>
+<span class="sourceLineNo">200</span>    for (int i = 0; i &lt; this.numTries; 
++i) {<a name="line.200"></a>
+<span class="sourceLineNo">201</span>      serverTrackerTimeout = 
serverTrackerTimeout + ConnectionUtils.getPauseTime(this.pause, i);<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>    this.rpcCallerFactory = rpcCaller;<a 
name="line.204"></a>
+<span class="sourceLineNo">205</span>    this.rpcFactory = rpcFactory;<a 
name="line.205"></a>
+<span class="sourceLineNo">206</span>    this.logBatchErrorDetails = 
conf.getBoolean(LOG_DETAILS_FOR_BATCH_ERROR, false);<a name="line.206"></a>
+<span class="sourceLineNo">207</span><a name="line.207"></a>
+<span class="sourceLineNo">208</span>    this.requestController = 
RequestControllerFactory.create(conf);<a name="line.208"></a>
+<span class="sourceLineNo">209</span>  }<a name="line.209"></a>
+<span class="sourceLineNo">210</span><a name="line.210"></a>
+<span class="sourceLineNo">211</span>  /**<a name="line.211"></a>
+<span class="sourceLineNo">212</span>   * The submitted task may be not 
accomplished at all if there are too many running tasks or<a 
name="line.212"></a>
+<span class="sourceLineNo">213</span>   * other limits.<a name="line.213"></a>
+<span class="sourceLineNo">214</span>   * @param &lt;CResult&gt; The class to 
cast the result<a name="line.214"></a>
+<span class="sourceLineNo">215</span>   * @param task The setting and data<a 
name="line.215"></a>
+<span class="sourceLineNo">216</span>   * @return AsyncRequestFuture<a 
name="line.216"></a>
+<span class="sourceLineNo">217</span>   */<a name="line.217"></a>
+<span class="sourceLineNo">218</span>  public &lt;CResult&gt; 
AsyncRequestFuture submit(AsyncProcessTask&lt;CResult&gt; task) throws 
InterruptedIOException {<a name="line.218"></a>
+<span class="sourceLineNo">219</span>    AsyncRequestFuture reqFuture = 
checkTask(task);<a name="line.219"></a>
+<span class="sourceLineNo">220</span>    if (reqFuture != null) {<a 
name="line.220"></a>
+<span class="sourceLineNo">221</span>      return reqFuture;<a 
name="line.221"></a>
+<span class="sourceLineNo">222</span>    }<a name="line.222"></a>
+<span class="sourceLineNo">223</span>    SubmittedRows submittedRows = 
task.getSubmittedRows() == null ? SubmittedRows.ALL : 
task.getSubmittedRows();<a name="line.223"></a>
+<span class="sourceLineNo">224</span>    switch (submittedRows) {<a 
name="line.224"></a>
+<span class="sourceLineNo">225</span>      case ALL:<a name="line.225"></a>
+<span class="sourceLineNo">226</span>        return submitAll(task);<a 
name="line.226"></a>
+<span class="sourceLineNo">227</span>      case AT_LEAST_ONE:<a 
name="line.227"></a>
+<span class="sourceLineNo">228</span>        return submit(task, true);<a 
name="line.228"></a>
+<span class="sourceLineNo">229</span>      default:<a name="line.229"></a>
+<span class="sourceLineNo">230</span>        return submit(task, false);<a 
name="line.230"></a>
+<span class="sourceLineNo">231</span>    }<a name="line.231"></a>
+<span class="sourceLineNo">232</span>  }<a name="line.232"></a>
+<span class="sourceLineNo">233</span><a name="line.233"></a>
+<span class="sourceLineNo">234</span>  /**<a name="line.234"></a>
+<span class="sourceLineNo">235</span>   * Extract from the rows list what we 
can submit. The rows we can not submit are kept in the<a name="line.235"></a>
+<span class="sourceLineNo">236</span>   * list. Does not send requests to 
replicas (not currently used for anything other<a name="line.236"></a>
+<span class="sourceLineNo">237</span>   * than streaming puts anyway).<a 
name="line.237"></a>
+<span class="sourceLineNo">238</span>   *<a name="line.238"></a>
+<span class="sourceLineNo">239</span>   * @param task The setting and data<a 
name="line.239"></a>
+<span class="sourceLineNo">240</span>   * @param atLeastOne true if we should 
submit at least a subset.<a name="line.240"></a>
+<span class="sourceLineNo">241</span>   */<a name="line.241"></a>
+<span class="sourceLineNo">242</span>  private &lt;CResult&gt; 
AsyncRequestFuture submit(AsyncProcessTask&lt;CResult&gt; task,<a 
name="line.242"></a>
+<span class="sourceLineNo">243</span>    boolean atLeastOne) throws 
InterruptedIOException {<a name="line.243"></a>
+<span class="sourceLineNo">244</span>    TableName tableName = 
task.getTableName();<a name="line.244"></a>
+<span class="sourceLineNo">245</span>    RowAccess&lt;? extends Row&gt; rows = 
task.getRowAccess();<a name="line.245"></a>
+<span class="sourceLineNo">246</span>    Map&lt;ServerName, MultiAction&gt; 
actionsByServer = new HashMap&lt;&gt;();<a name="line.246"></a>
+<span class="sourceLineNo">247</span>    List&lt;Action&gt; retainedActions = 
new ArrayList&lt;&gt;(rows.size());<a name="line.247"></a>
+<span class="sourceLineNo">248</span><a name="line.248"></a>
+<span class="sourceLineNo">249</span>    NonceGenerator ng = 
this.connection.getNonceGenerator();<a name="line.249"></a>
+<span class="sourceLineNo">250</span>    long nonceGroup = ng.getNonceGroup(); 
// Currently, nonce group is per entire client.<a name="line.250"></a>
+<span class="sourceLineNo">251</span><a name="line.251"></a>
+<span class="sourceLineNo">252</span>    // Location errors that happen before 
we decide what requests to take.<a name="line.252"></a>
+<span class="sourceLineNo">253</span>    List&lt;Exception&gt; locationErrors 
= null;<a name="line.253"></a>
+<span class="sourceLineNo">254</span>    List&lt;Integer&gt; locationErrorRows 
= null;<a name="line.254"></a>
+<span class="sourceLineNo">255</span>    RequestController.Checker checker = 
requestController.newChecker();<a name="line.255"></a>
+<span class="sourceLineNo">256</span>    boolean firstIter = true;<a 
name="line.256"></a>
+<span class="sourceLineNo">257</span>    do {<a name="line.257"></a>
+<span class="sourceLineNo">258</span>      // Wait until there is at least one 
slot for a new task.<a name="line.258"></a>
+<span class="sourceLineNo">259</span>      
requestController.waitForFreeSlot(id, periodToLog, getLogger(tableName, -1));<a 
name="line.259"></a>
+<span class="sourceLineNo">260</span>      int posInList = -1;<a 
name="line.260"></a>
+<span class="sourceLineNo">261</span>      if (!firstIter) {<a 
name="line.261"></a>
+<span class="sourceLineNo">262</span>        checker.reset();<a 
name="line.262"></a>
+<span class="sourceLineNo">263</span>      }<a name="line.263"></a>
+<span class="sourceLineNo">264</span>      Iterator&lt;? extends Row&gt; it = 
rows.iterator();<a name="line.264"></a>
+<span class="sourceLineNo">265</span>      while (it.hasNext()) {<a 
name="line.265"></a>
+<span class="sourceLineNo">266</span>        Row r = it.next();<a 
name="line.266"></a>
+<span class="sourceLineNo">267</span>        HRegionLocation loc;<a 
name="line.267"></a>
+<span class="sourceLineNo">268</span>        try {<a name="line.268"></a>
+<span class="sourceLineNo">269</span>          if (r == null) {<a 
name="line.269"></a>
+<span class="sourceLineNo">270</span>            throw new 
IllegalArgumentException("#" + id + ", row cannot be null");<a 
name="line.270"></a>
+<span class="sourceLineNo">271</span>          }<a name="line.271"></a>
+<span class="sourceLineNo">272</span>          // Make sure we get 0-s 
replica.<a name="line.272"></a>
+<span class="sourceLineNo">273</span>          RegionLocations locs = 
connection.locateRegion(<a name="line.273"></a>
+<span class="sourceLineNo">274</span>              tableName, r.getRow(), 
true, true, RegionReplicaUtil.DEFAULT_REPLICA_ID);<a name="line.274"></a>
+<span class="sourceLineNo">275</span>          if (locs == null || 
locs.isEmpty() || locs.getDefaultRegionLocation() == null) {<a 
name="line.275"></a>
+<span class="sourceLineNo">276</span>            throw new IOException("#" + 
id + ", no location found, aborting submit for"<a name="line.276"></a>
+<span class="sourceLineNo">277</span>                + " tableName=" + 
tableName + " rowkey=" + Bytes.toStringBinary(r.getRow()));<a 
name="line.277"></a>
+<span class="sourceLineNo">278</span>          }<a name="line.278"></a>
+<span class="sourceLineNo">279</span>          loc = 
locs.getDefaultRegionLocation();<a name="line.279"></a>
+<span class="sourceLineNo">280</span>        } catch (IOException ex) {<a 
name="line.280"></a>
+<span class="sourceLineNo">281</span>          locationErrors = new 
ArrayList&lt;&gt;(1);<a name="line.281"></a>
+<span class="sourceLineNo">282</span>          locationErrorRows = new 
ArrayList&lt;&gt;(1);<a name="line.282"></a>
+<span class="sourceLineNo">283</span>          LOG.error("Failed to get region 
location ", ex);<a name="line.283"></a>
+<span class="sourceLineNo">284</span>          // This action failed before 
creating ars. Retain it, but do not add to submit list.<a name="line.284"></a>
+<span class="sourceLineNo">285</span>          // We will then add it to ars 
in an already-failed state.<a name="line.285"></a>
+<span class="sourceLineNo">286</span><a name="line.286"></a>
+<span class="sourceLineNo">287</span>          int priority = 
HConstants.NORMAL_QOS;<a name="line.287"></a>
+<span class="sourceLineNo">288</span>          if (r instanceof Mutation) {<a 
name="line.288"></a>
+<span class="sourceLineNo">289</span>            priority = ((Mutation) 
r).getPriority();<a name="line.289"></a>
+<span class="sourceLineNo">290</span>          }<a name="line.290"></a>
+<span class="sourceLineNo">291</span>          retainedActions.add(new 
Action(r, ++posInList, priority));<a name="line.291"></a>
+<span class="sourceLineNo">292</span>          locationErrors.add(ex);<a 
name="line.292"></a>
+<span class="sourceLineNo">293</span>          
locationErrorRows.add(posInList);<a name="line.293"></a>
+<span class="sourceLineNo">294</span>          it.remove();<a 
name="line.294"></a>
+<span class="sourceLineNo">295</span>          break; // Backward compat: we 
stop considering actions on location error.<a name="line.295"></a>
+<span class="sourceLineNo">296</span>        }<a name="line.296"></a>
+<span class="sourceLineNo">297</span>        ReturnCode code = 
checker.canTakeRow(loc, r);<a name="line.297"></a>
+<span class="sourceLineNo">298</span>        if (code == ReturnCode.END) {<a 
name="line.298"></a>
+<span class="sourceLineNo">299</span>          break;<a name="line.299"></a>
+<span class="sourceLineNo">300</span>        }<a name="line.300"></a>
+<span class="sourceLineNo">301</span>        if (code == ReturnCode.INCLUDE) 
{<a name="line.301"></a>
+<span class="sourceLineNo">302</span>          int priority = 
HConstants.NORMAL_QOS;<a name="line.302"></a>
+<span class="sourceLineNo">303</span>          if (r instanceof Mutation) {<a 
name="line.303"></a>
+<span class="sourceLineNo">304</span>            priority = ((Mutation) 
r).getPriority();<a name="line.304"></a>
+<span class="sourceLineNo">305</span>          }<a name="line.305"></a>
+<span class="sourceLineNo">306</span>          Action action = new Action(r, 
++posInList, priority);<a name="line.306"></a>
+<span class="sourceLineNo">307</span>          setNonce(ng, r, action);<a 
name="line.307"></a>
+<span class="sourceLineNo">308</span>          retainedActions.add(action);<a 
name="line.308"></a>
+<span class="sourceLineNo">309</span>          // TODO: replica-get is not 
supported on this path<a name="line.309"></a>
+<span class="sourceLineNo">310</span>          byte[] regionName = 
loc.getRegionInfo().getRegionName();<a name="line.310"></a>
+<span class="sourceLineNo">311</span>          addAction(loc.getServerName(), 
regionName, action, actionsByServer, nonceGroup);<a name="line.311"></a>
+<span class="sourceLineNo">312</span>          it.remove();<a 
name="line.312"></a>
+<span class="sourceLineNo">313</span>        }<a name="line.313"></a>
+<span class="sourceLineNo">314</span>      }<a name="line.314"></a>
+<span class="sourceLineNo">315</span>      firstIter = false;<a 
name="line.315"></a>
+<span class="sourceLineNo">316</span>    } while (retainedActions.isEmpty() 
&amp;&amp; atLeastOne &amp;&amp; (locationErrors == null));<a 
name="line.316"></a>
+<span class="sourceLineNo">317</span><a name="line.317"></a>
+<span class="sourceLineNo">318</span>    if (retainedActions.isEmpty()) return 
NO_REQS_RESULT;<a name="line.318"></a>
+<span class="sourceLineNo">319</span><a name="line.319"></a>
+<span class="sourceLineNo">320</span>    return submitMultiActions(task, 
retainedActions, nonceGroup,<a name="line.320"></a>
+<span class="sourceLineNo">321</span>        locationErrors, 
locationErrorRows, actionsByServer);<a name="line.321"></a>
+<span class="sourceLineNo">322</span>  }<a name="line.322"></a>
+<span class="sourceLineNo">323</span><a name="line.323"></a>
+<span class="sourceLineNo">324</span>  &lt;CResult&gt; AsyncRequestFuture 
submitMultiActions(AsyncProcessTask task,<a name="line.324"></a>
+<span class="sourceLineNo">325</span>      List&lt;Action&gt; retainedActions, 
long nonceGroup, List&lt;Exception&gt; locationErrors,<a name="line.325"></a>
+<span class="sourceLineNo">326</span>      List&lt;Integer&gt; 
locationErrorRows, Map&lt;ServerName, MultiAction&gt; actionsByServer) {<a 
name="line.326"></a>
+<span class="sourceLineNo">327</span>    AsyncRequestFutureImpl&lt;CResult&gt; 
ars = createAsyncRequestFuture(task, retainedActions, nonceGroup);<a 
name="line.327"></a>
+<span class="sourceLineNo">328</span>    // Add location errors if any<a 
name="line.328"></a>
+<span class="sourceLineNo">329</span>    if (locationErrors != null) {<a 
name="line.329"></a>
+<span class="sourceLineNo">330</span>      for (int i = 0; i &lt; 
locationErrors.size(); ++i) {<a name="line.330"></a>
+<span class="sourceLineNo">331</span>        int originalIndex = 
locationErrorRows.get(i);<a name="line.331"></a>
+<span class="sourceLineNo">332</span>        Row row = 
retainedActions.get(originalIndex).getAction();<a name="line.332"></a>
+<span class="sourceLineNo">333</span>        ars.manageError(originalIndex, 
row,<a name="line.333"></a>
+<span class="sourceLineNo">334</span>            
AsyncRequestFutureImpl.Retry.NO_LOCATION_PROBLEM, locationErrors.get(i), 
null);<a name="line.334"></a>
+<span class="sourceLineNo">335</span>      }<a name="line.335"></a>
+<span class="sourceLineNo">336</span>    }<a name="line.336"></a>
+<span class="sourceLineNo">337</span>    ars.sendMultiAction(actionsByServer, 
1, null, false);<a name="line.337"></a>
+<span class="sourceLineNo">338</span>    return ars;<a name="line.338"></a>
+<span class="sourceLineNo">339</span>  }<a name="line.339"></a>
+<span class="sourceLineNo">340</span><a name="line.340"></a>
+<span class="sourceLineNo">341</span>  /**<a name="line.341"></a>
+<span class="sourceLineNo">342</span>   * Helper that is used when grouping 
the actions per region server.<a name="line.342"></a>
+<span class="sourceLineNo">343</span>   *<a name="line.343"></a>
+<span class="sourceLineNo">344</span>   * @param server - server<a 
name="line.344"></a>
+<span class="sourceLineNo">345</span>   * @param regionName - regionName<a 
name="line.345"></a>
+<span class="sourceLineNo">346</span>   * @param action - the action to add to 
the multiaction<a name="line.346"></a>
+<span class="sourceLineNo">347</span>   * @param actionsByServer the 
multiaction per server<a name="line.347"></a>
+<span class="sourceLineNo">348</span>   * @param nonceGroup Nonce group.<a 
name="line.348"></a>
+<span class="sourceLineNo">349</span>   */<a name="line.349"></a>
+<span class="sourceLineNo">350</span>  static void addAction(ServerName 
server, byte[] regionName, Action action,<a name="line.350"></a>
+<span class="sourceLineNo">351</span>      Map&lt;ServerName, MultiAction&gt; 
actionsByServer, long nonceGroup) {<a name="line.351"></a>
+<span class="sourceLineNo">352</span>    MultiAction multiAction = 
actionsByServer.get(server);<a name="line.352"></a>
+<span class="sourceLineNo">353</span>    if (multiAction == null) {<a 
name="line.353"></a>
+<span class="sourceLineNo">354</span>      multiAction = new MultiAction();<a 
name="line.354"></a>
+<span class="sourceLineNo">355</span>      actionsByServer.put(server, 
multiAction);<a name="line.355"></a>
+<span class="sourceLineNo">356</span>    }<a name="line.356"></a>
+<span class="sourceLineNo">357</span>    if (action.hasNonce() &amp;&amp; 
!multiAction.hasNonceGroup()) {<a name="line.357"></a>
+<span class="sourceLineNo">358</span>      
multiAction.setNonceGroup(nonceGroup);<a name="line.358"></a>
+<span class="sourceLineNo">359</span>    }<a name="line.359"></a>
+<span class="sourceLineNo">360</span><a name="line.360"></a>
+<span class="sourceLineNo">361</span>    multiAction.add(regionName, 
action);<a name="line.361"></a>
+<span class="sourceLineNo">362</span>  }<a name="line.362"></a>
+<span class="sourceLineNo">363</span><a name="line.363"></a>
+<span class="sourceLineNo">364</span>  /**<a name="line.364"></a>
+<span class="sourceLineNo">365</span>   * Submit immediately the list of rows, 
whatever the server status. Kept for backward<a name="line.365"></a>
+<span class="sourceLineNo">366</span>   * compatibility: it allows to be used 
with the batch interface that return an array of objects.<a name="line.366"></a>
+<span class="sourceLineNo">367</span>   * @param task The setting and data<a 
name="line.367"></a>
+<span class="sourceLineNo">368</span>   */<a name="line.368"></a>
+<span class="sourceLineNo">369</span>  private &lt;CResult&gt; 
AsyncRequestFuture submitAll(AsyncProcessTask task) {<a name="line.369"></a>
+<span class="sourceLineNo">370</span>    RowAccess&lt;? extends Row&gt; rows = 
task.getRowAccess();<a name="line.370"></a>
+<span class="sourceLineNo">371</span>    List&lt;Action&gt; actions = new 
ArrayList&lt;&gt;(rows.size());<a name="line.371"></a>
+<span class="sourceLineNo">372</span><a name="line.372"></a>
+<span class="sourceLineNo">373</span>    // The position will be used by the 
processBatch to match the object array returned.<a name="line.373"></a>
+<span class="sourceLineNo">374</span>    int posInList = -1;<a 
name="line.374"></a>
+<span class="sourceLineNo">375</span>    NonceGenerator ng = 
this.connection.getNonceGenerator();<a name="line.375"></a>
+<span class="sourceLineNo">376</span>    int highestPriority = 
HConstants.PRIORITY_UNSET;<a name="line.376"></a>
+<span class="sourceLineNo">377</span>    for (Row r : rows) {<a 
name="line.377"></a>
+<span class="sourceLineNo">378</span>      posInList++;<a name="line.378"></a>
+<span class="sourceLineNo">379</span>      if (r instanceof Put) {<a 
name="line.379"></a>
+<span class="sourceLineNo">380</span>        Put put = (Put) r;<a 
name="line.380"></a>
+<span class="sourceLineNo">381</span>        if (put.isEmpty()) {<a 
name="line.381"></a>
+<span class="sourceLineNo">382</span>          throw new 
IllegalArgumentException("No columns to insert for #" + (posInList+1)+ " 
item");<a name="line.382"></a>
+<span class="sourceLineNo">383</span>        }<a name="line.383"></a>
+<span class="sourceLineNo">384</span>        highestPriority = 
Math.max(put.getPriority(), highestPriority);<a name="line.384"></a>
+<span class="sourceLineNo">385</span>      }<a name="line.385"></a>
+<span class="sourceLineNo">386</span>      Action action = new Action(r, 
posInList, highestPriority);<a name="line.386"></a>
+<span class="sourceLineNo">387</span>      setNonce(ng, r, action);<a 
name="line.387"></a>
+<span class="sourceLineNo">388</span>      actions.add(action);<a 
name="line.388"></a>
+<span class="sourceLineNo">389</span>    }<a name="line.389"></a>
+<span class="sourceLineNo">390</span>    AsyncRequestFutureImpl&lt;CResult&gt; 
ars = createAsyncRequestFuture(task, actions, ng.getNonceGroup());<a 
name="line.390"></a>
+<span class="sourceLineNo">391</span>    ars.groupAndSendMultiAction(actions, 
1);<a name="line.391"></a>
+<span class="sourceLineNo">392</span>    return ars;<a name="line.392"></a>
+<span class="sourceLineNo">393</span>  }<a name="line.393"></a>
+<span class="sourceLineNo">394</span><a name="line.394"></a>
+<span class="sourceLineNo">395</span>  private &lt;CResult&gt; 
AsyncRequestFuture checkTask(AsyncProcessTask&lt;CResult&gt; task) {<a 
name="line.395"></a>
+<span class="sourceLineNo">396</span>    if (task.getRowAccess() == null || 
task.getRowAccess().isEmpty()) {<a name="line.396"></a>
+<span class="sourceLineNo">397</span>      return NO_REQS_RESULT;<a 
name="line.397"></a>
+<span class="sourceLineNo">398</span>    }<a name="line.398"></a>
+<span class="sourceLineNo">399</span>    
Objects.requireNonNull(task.getPool(), "The pool can't be NULL");<a 
name="line.399"></a>
+<span class="sourceLineNo">400</span>    
checkOperationTimeout(task.getOperationTimeout());<a name="line.400"></a>
+<span class="sourceLineNo">401</span>    
checkRpcTimeout(task.getRpcTimeout());<a name="line.401"></a>
+<span class="sourceLineNo">402</span>    return null;<a name="line.402"></a>
+<span class="sourceLineNo">403</span>  }<a name="line.403"></a>
+<span class="sourceLineNo">404</span><a name="line.404"></a>
+<span class="sourceLineNo">405</span>  private void setNonce(NonceGenerator 
ng, Row r, Action action) {<a name="line.405"></a>
+<span class="sourceLineNo">406</span>    if (!(r instanceof Append) &amp;&amp; 
!(r instanceof Increment)) return;<a name="line.406"></a>
+<span class="sourceLineNo">407</span>    action.setNonce(ng.newNonce()); // 
Action handles NO_NONCE, so it's ok if ng is disabled.<a name="line.407"></a>
+<span class="sourceLineNo">408</span>  }<a name="line.408"></a>
+<span class="sourceLineNo">409</span><a name="line.409"></a>
+<span class="sourceLineNo">410</span>  private int checkTimeout(String name, 
int timeout) {<a name="line.410"></a>
+<span class="sourceLineNo">411</span>    if (timeout &lt; 0) {<a 
name="line.411"></a>
+<span class="sourceLineNo">412</span>      throw new RuntimeException("The " + 
name + " must be bigger than zero,"<a name="line.412"></a>
+<span class="sourceLineNo">413</span>        + "current value is" + 
timeout);<a name="line.413"></a>
+<span class="sourceLineNo">414</span>    }<a name="line.414"></a>
+<span class="sourceLineNo">415</span>    return timeout;<a name="line.415"></a>
 <span class="sourceLineNo">416</span>  }<a name="line.416"></a>
-<span class="sourceLineNo">417</span><a name="line.417"></a>
-<span class="sourceLineNo">418</span>  private int checkTimeout(String name, 
int timeout) {<a name="line.418"></a>
-<span class="sourceLineNo">419</span>    if (timeout &lt; 0) {<a 
name="line.419"></a>
-<span class="sourceLineNo">420</span>      throw new RuntimeException("The " + 
name + " must be bigger than zero,"<a name="line.420"></a>
-<span class="sourceLineNo">421</span>        + "current value is" + 
timeout);<a name="line.421"></a>
-<span class="sourceLineNo">422</span>    }<a name="line.422"></a>
-<span class="sourceLineNo">423</span>    return timeout;<a name="line.423"></a>
-<span class="sourceLineNo">424</span>  }<a name="line.424"></a>
-<span class="sourceLineNo">425</span>  private int checkOperationTimeout(int 
operationTimeout) {<a name="line.425"></a>
-<span class="sourceLineNo">426</span>    return checkTimeout("operation 
timeout", operationTimeout);<a name="line.426"></a>
-<span class="sourceLineNo">427</span>  }<a name="line.427"></a>
-<span class="sourceLineNo">428</span><a name="line.428"></a>
-<span class="sourceLineNo">429</span>  private int checkRpcTimeout(int 
rpcTimeout) {<a name="line.429"></a>
-<span class="sourceLineNo">430</span>    return checkTimeout("rpc timeout", 
rpcTimeout);<a name="line.430"></a>
-<span class="sourceLineNo">431</span>  }<a name="line.431"></a>
-<span class="sourceLineNo">432</span><a name="line.432"></a>
-<span class="sourceLineNo">433</span>  @VisibleForTesting<a 
name="line.433"></a>
-<span class="sourceLineNo">434</span>  &lt;CResult&gt; 
AsyncRequestFutureImpl&lt;CResult&gt; createAsyncRequestFuture(<a 
name="line.434"></a>
-<span class="sourceLineNo">435</span>      AsyncProcessTask task, 
List&lt;Action&gt; actions, long nonceGroup) {<a name="line.435"></a>
-<span class="sourceLineNo">436</span>    return new 
AsyncRequestFutureImpl&lt;&gt;(task, actions, nonceGroup, this);<a 
name="line.436"></a>
-<span class="sourceLineNo">437</span>  }<a name="line.437"></a>
-<span class="sourceLineNo">438</span><a name="line.438"></a>
-<span class="sourceLineNo">439</span>  /** Wait until the async does not have 
more than max tasks in progress. */<a name="line.439"></a>
-<span class="sourceLineNo">440</span>  protected void 
waitForMaximumCurrentTasks(int max, TableName tableName)<a name="line.440"></a>
-<span class="sourceLineNo">441</span>      throws InterruptedIOException {<a 
name="line.441"></a>
-<span class="sourceLineNo">442</span>    
requestController.waitForMaximumCurrentTasks(max, id, periodToLog,<a 
name="line.442"></a>
-<span class="sourceLineNo">443</span>      getLogger(tableName, max));<a 
name="line.443"></a>
-<span class="sourceLineNo">444</span>  }<a name="line.444"></a>
-<span class="sourceLineNo">445</span><a name="line.445"></a>
-<span class="sourceLineNo">446</span>  private Consumer&lt;Long&gt; 
getLogger(TableName tableName, long max) {<a name="line.446"></a>
-<span class="sourceLineNo">447</span>    return (currentInProgress) -&gt; {<a 
name="line.447"></a>
-<span class="sourceLineNo">448</span>      LOG.info("#" + id + (max &lt; 0 ? 
", waiting for any free slot"<a name="line.448"></a>
-<span class="sourceLineNo">449</span>      : ", waiting for some tasks to 
finish. Expected max="<a name="line.449"></a>
-<span class="sourceLineNo">450</span>      + max) + ", tasksInProgress=" + 
currentInProgress +<a name="line.450"></a>
-<span class="sourceLineNo">451</span>      " hasError=" + hasError() + 
(tableName == null ? "" : ", tableName=" + tableName));<a name="line.451"></a>
-<span class="sourceLineNo">452</span>    };<a name="line.452"></a>
-<span class="sourceLineNo">453</span>  }<a name="line.453"></a>
-<span class="sourceLineNo">454</span><a name="line.454"></a>
-<span class="sourceLineNo">455</span>  void 
incTaskCounters(Collection&lt;byte[]&gt; regions, ServerName sn) {<a 
name="line.455"></a>
-<span class="sourceLineNo">456</span>    
requestController.incTaskCounters(regions, sn);<a name="line.456"></a>
-<span class="sourceLineNo">457</span>  }<a name="line.457"></a>
-<span class="sourceLineNo">458</span><a name="line.458"></a>
-<span class="sourceLineNo">459</span><a name="line.459"></a>
-<span class="sourceLineNo">460</span>  void 
decTaskCounters(Collection&lt;byte[]&gt; regions, ServerName sn) {<a 
name="line.460"></a>
-<span class="sourceLineNo">461</span>    
requestController.decTaskCounters(regions, sn);<a name="line.461"></a>
-<span class="sourceLineNo">462</span>  }<a name="line.462"></a>
-<span class="sourceLineNo">463</span>  /**<a name="line.463"></a>
-<span class="sourceLineNo">464</span>   * Only used w/useGlobalErrors ctor 
argument, for HTable backward compat.<a name="line.464"></a>
-<span class="sourceLineNo">465</span>   * @return Whether there were any 
errors in any request since the last time<a name="line.465"></a>
-<span class="sourceLineNo">466</span>   *          {@link 
#waitForAllPreviousOpsAndReset(List, TableName)} was called, or AP was 
created.<a name="line.466"></a>
-<span class="sourceLineNo">467</span>   */<a name="line.467"></a>
-<span class="sourceLineNo">468</span>  public boolean hasError() {<a 
name="line.468"></a>
-<span class="sourceLineNo">469</span>    return globalErrors != null 
&amp;&amp; globalErrors.hasErrors();<a name="line.469"></a>
-<span class="sourceLineNo">470</span>  }<a name="line.470"></a>
-<span class="sourceLineNo">471</span><a name="line.471"></a>
-<span class="sourceLineNo">472</span>  /**<a name="line.472"></a>
-<span class="sourceLineNo">473</span>   * Only used w/useGlobalErrors ctor 
argument, for HTable backward compat.<a name="line.473"></a>
-<span class="sourceLineNo">474</span>   * Waits for all previous operations to 
finish, and returns errors and (optionally)<a name="line.474"></a>
-<span class="sourceLineNo">475</span>   * failed operations themselves.<a 
name="line.475"></a>
-<span class="sourceLineNo">476</span>   * @param failedRows an optional list 
into which the rows that failed since the last time<a name="line.476"></a>
-<span class="sourceLineNo">477</span>   *        {@link 
#waitForAllPreviousOpsAndReset(List, TableName)} was called, or AP was created, 
are saved.<a name="line.477"></a>
-<span class="sourceLineNo">478</span>   * @param tableName name of the table<a 
name="line.478"></a>
-<span class="sourceLineNo">479</span>   * @return all the errors since the 
last time {@link #waitForAllPreviousOpsAndReset(List, TableName)}<a 
name="line.479"></a>
-<span class="sourceLineNo">480</span>   *          was called, or AP was 
created.<a name="line.480"></a>
-<span class="sourceLineNo">481</span>   */<a name="line.481"></a>
-<span class="sourceLineNo">482</span>  public 
RetriesExhaustedWithDetailsException waitForAllPreviousOpsAndReset(<a 
name="line.482"></a>
-<span class="sourceLineNo">483</span>      List&lt;Row&gt; failedRows, 
TableName tableName) throws InterruptedIOException {<a name="line.483"></a>
-<span class="sourceLineNo">484</span>    waitForMaximumCurrentTasks(0, 
tableName);<a name="line.484"></a>
-<span class="sourceLineNo">485</span>    if (globalErrors == null || 
!globalErrors.hasErrors()) {<a name="line.485"></a>
-<span class="sourceLineNo">486</span>      return null;<a name="line.486"></a>
-<span class="sourceLineNo">487</span>    }<a name="line.487"></a>
-<span class="sourceLineNo">488</span>    if (failedRows != null) {<a 
name="line.488"></a>
-<span class="sourceLineNo">489</span>      
failedRows.addAll(globalErrors.actions);<a name="line.489"></a>
-<span class="sourceLineNo">490</span>    }<a name="line.490"></a>
-<span class="sourceLineNo">491</span>    RetriesExhaustedWithDetailsException 
result = globalErrors.makeException(logBatchErrorDetails);<a 
name="line.491"></a>
-<span class="sourceLineNo">492</span>    globalErrors.clear();<a 
name="line.492"></a>
-<span class="sourceLineNo">493</span>    return result;<a name="line.493"></a>
-<span class="sourceLineNo">494</span>  }<a name="line.494"></a>
-<span class="sourceLineNo">495</span><a name="line.495"></a>
-<span class="sourceLineNo">496</span>  /**<a name="line.496"></a>
-<span class="sourceLineNo">497</span>   * Create a caller. Isolated to be 
easily overridden in the tests.<a name="line.497"></a>
-<span class="sourceLineNo">498</span>   */<a name="line.498"></a>
-<span class="sourceLineNo">499</span>  @VisibleForTesting<a 
name="line.499"></a>
-<span class="sourceLineNo">500</span>  protected 
RpcRetryingCaller&lt;AbstractResponse&gt; createCaller(<a name="line.500"></a>
-<span class="sourceLineNo">501</span>      CancellableRegionServerCallable 
callable, int rpcTimeout) {<a name="line.501"></a>
-<span class="sourceLineNo">502</span>    return 
rpcCallerFactory.&lt;AbstractResponse&gt; 
newCaller(checkRpcTimeout(rpcTimeout));<a name="line.502"></a>
-<span class="sourceLineNo">503</span>  }<a name="line.503"></a>
-<span class="sourceLineNo">504</span><a name="line.504"></a>
-<span class="sourceLineNo">505</span><a name="line.505"></a>
-<span class="sourceLineNo">506</span>  /**<a name="line.506"></a>
-<span class="sourceLineNo">507</span>   * Creates the server error tracker to 
use inside process.<a name="line.507"></a>
-<span class="sourceLineNo">508</span>   * Currently, to preserve the main 
assumption about current retries, and to work well with<a name="line.508"></a>
-<span class="sourceLineNo">509</span>   * the retry-limit-based calculation, 
the calculation is local per Process object.<a name="line.509"></a>
-<span class="sourceLineNo">510</span>   * We may benefit from connection-wide 
tracking of server errors.<a name="line.510"></a>
-<span class="sourceLineNo">511</span>   * @return ServerErrorTracker to use, 
null if there is no ServerErrorTracker on this connection<a name="line.511"></a>
-<span class="sourceLineNo">512</span>   */<a name="line.512"></a>
-<span class="sourceLineNo">513</span>  
ConnectionImplementation.ServerErrorTracker createServerErrorTracker() {<a 
name="line.513"></a>
-<span class="sourceLineNo">514</span>    return new 
ConnectionImplementation.ServerErrorTracker(<a name="line.514"></a>
-<span class="sourceLineNo">515</span>        this.serverTrackerTimeout, 
this.numTries);<a name="line.515"></a>
-<span class="sourceLineNo">516</span>  }<a name="line.516"></a>
-<span class="sourceLineNo">517</span><a name="line.517"></a>
-<span class="sourceLineNo">518</span>  static boolean isReplicaGet(Row row) 
{<a name="line.518"></a>
-<span class="sourceLineNo">519</span>    return (row instanceof Get) 
&amp;&amp; (((Get)row).getConsistency() == Consistency.TIMELINE);<a 
name="line.519"></a>
-<span class="sourceLineNo">520</span>  }<a name="line.520"></a>
-<span class="sourceLineNo">521</span><a name="line.521"></a>
-<span class="sourceLineNo">522</span>}<a name="line.522"></a>
+<span class="sourceLineNo">417</span>  private int checkOperationTimeout(int 
operationTimeout) {<a name="line.417"></a>
+<span class="sourceLineNo">418</span>    return checkTimeout("operation 
timeout", operationTimeout);<a name="line.418"></a>
+<span class="sourceLineNo">419</span>  }<a name="line.419"></a>
+<span class="sourceLineNo">420</span><a name="line.420"></a>
+<span class="sourceLineNo">421</span>  private int checkRpcTimeout(int 
rpcTimeout) {<a name="line.421"></a>
+<span class="sourceLineNo">422</span>    return checkTimeout("rpc timeout", 
rpcTimeout);<a name="line.422"></a>
+<span class="sourceLineNo">423</span>  }<a name="line.423"></a>
+<span class="sourceLineNo">424</span><a name="line.424"></a>
+<span class="sourceLineNo">425</span>  @VisibleForTesting<a 
name="line.425"></a>
+<span class="sourceLineNo">426</span>  &lt;CResult&gt; 
AsyncRequestFutureImpl&lt;CResult&gt; createAsyncRequestFuture(<a 
name="line.426"></a>
+<span class="sourceLineNo">427</span>      AsyncProcessTask task, 
List&lt;Action&gt; actions, long nonceGroup) {<a name="line.427"></a>
+<span class="sourceLineNo">428</span>    return new 
AsyncRequestFutureImpl&lt;&gt;(task, actions, nonceGroup, this);<a 
name="line.428"></a>
+<span class="sourceLineNo">429</span>  }<a name="line.429"></a>
+<span class="sourceLineNo">430</span><a name="line.430"></a>
+<span class="sourceLineNo">431</span>  /** Wait until the async does not have 
more than max tasks in progress. */<a name="line.431"></a>
+<span class="sourceLineNo">432</span>  protected void 
waitForMaximumCurrentTasks(int max, TableName tableName)<a name="line.432"></a>
+<span class="sourceLineNo">433</span>      throws InterruptedIOException {<a 
name="line.433"></a>
+<span class="sourceLineNo">434</span>    
requestController.waitForMaximumCurrentTasks(max, id, periodToLog,<a 
name="line.434"></a>
+<span class="sourceLineNo">435</span>      getLogger(tableName, max));<a 
name="line.435"></a>
+<span class="sourceLineNo">436</span>  }<a name="line.436"></a>
+<span class="sourceLineNo">437</span><a name="line.437"></a>
+<span class="sourceLineNo">438</span>  private Consumer&lt;Long&gt; 
getLogger(TableName tableName, long max) {<a name="line.438"></a>
+<span class="sourceLineNo">439</span>    return (currentInProgress) -&gt; {<a 
name="line.439"></a>
+<span class="sourceLineNo">440</span>      LOG.info("#" + id + (max &lt; 0 ?<a 
name="line.440"></a>
+<span class="sourceLineNo">441</span>          ", waiting for any free slot" 
:<a name="line.441"></a>
+<span class="sourceLineNo">442</span>          ", waiting for some tasks to 
finish. Expected max=" + max) + ", tasksInProgress="<a name="line.442"></a>
+<span class="sourceLineNo">443</span>          + currentInProgress + 
(tableName == null ? "" : ", tableName=" + tableName));<a name="line.443"></a>
+<span class="sourceLineNo">444</span>    };<a name="line.444"></a>
+<span class="sourceLineNo">445</span>  }<a name="line.445"></a>
+<span class="sourceLineNo">446</span><a name="line.446"></a>
+<span class="sourceLineNo">447</span>  void 
incTaskCounters(Collection&lt;byte[]&gt; regions, ServerName sn) {<a 
name="line.447"></a>
+<span class="sourceLineNo">448</span>    
requestController.incTaskCounters(regions, sn);<a name="line.448"></a>
+<span class="sourceLineNo">449</span>  }<a name="line.449"></a>
+<span class="sourceLineNo">450</span><a name="line.450"></a>
+<span class="sourceLineNo">451</span><a name="line.451"></a>
+<span class="sourceLineNo">452</span>  void 
decTaskCounters(Collection&lt;byte[]&gt; regions, ServerName sn) {<a 
name="line.452"></a>
+<span class="sourceLineNo">453</span>    
requestController.decTaskCounters(regions, sn);<a name="line.453"></a>
+<span class="sourceLineNo">454</span>  }<a name="line.454"></a>
+<span class="sourceLineNo">455</span><a name="line.455"></a>
+<span class="sourceLineNo">456</span>  /**<a name="line.456"></a>
+<span class="sourceLineNo">457</span>   * Create a caller. Isolated to be 
easily overridden in the tests.<a name="line.457"></a>
+<span class="sourceLineNo">458</span>   */<a name="line.458"></a>
+<span class="sourceLineNo">459</span>  @VisibleForTesting<a 
name="line.459"></a>
+<span class="sourceLineNo">460</span>  protected 
RpcRetryingCaller&lt;AbstractResponse&gt; createCaller(<a name="line.460"></a>
+<span class="sourceLineNo">461</span>      CancellableRegionServerCallable 
callable, int rpcTimeout) {<a name="line.461"></a>
+<span class="sourceLineNo">462</span>    return 
rpcCallerFactory.&lt;AbstractResponse&gt; 
newCaller(checkRpcTimeout(rpcTimeout));<a name="line.462"></a>
+<span class="sourceLineNo">463</span>  }<a name="line.463"></a>
+<span class="sourceLineNo">464</span><a name="line.464"></a>
+<span class="sourceLineNo">465</span><a name="line.465"></a>
+<span class="sourceLineNo">466</span>  /**<a name="line.466"></a>
+<span class="sourceLineNo">467</span>   * Creates the server error tracker to 
use inside process.<a name="line.467"></a>
+<span class="sourceLineNo">468</span>   * Currently, to preserve the main 
assumption about current retries, and to work well with<a name="line.468"></a>
+<span class="sourceLineNo">469</span>   * the retry-limit-based calculation, 
the calculation is local per Process object.<a name="line.469"></a>
+<span class="sourceLineNo">470</span>   * We may benefit from connection-wide 
tracking of server errors.<a name="line.470"></a>
+<span class="sourceLineNo">471</span>   * @return ServerErrorTracker to use, 
null if there is no ServerErrorTracker on this connection<a name="line.471"></a>
+<span class="sourceLineNo">472</span>   */<a name="line.472"></a>
+<span class="sourceLineNo">473</span>  
ConnectionImplementation.ServerErrorTracker createServerErrorTracker() {<a 
name="line.473"></a>
+<span class="sourceLineNo">474</span>    return new 
ConnectionImplementation.ServerErrorTracker(<a name="line.474"></a>
+<span class="sourceLineNo">475</span>        this.serverTrackerTimeout, 
this.numTries);<a name="line.475"></a>
+<span class="sourceLineNo">476</span>  }<a name="line.476"></a>
+<span class="sourceLineNo">477</span><a name="line.477"></a>
+<span class="sourceLineNo">478</span>  static boolean isReplicaGet(Row row) 
{<a name="line.478"></a>
+<span class="sourceLineNo">479</span>    return (row instanceof Get) 
&amp;&amp; (((Get)row).getConsistency() == Consistency.TIMELINE);<a 
name="line.479"></a>
+<span class="sourceLineNo">480</span>  }<a name="line.480"></a>
+<span class="sourceLineNo">481</span><a name="line.481"></a>
+<span class="sourceLineNo">482</span>}<a name="line.482"></a>
 
 
 

Reply via email to