You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by gi...@apache.org on 2018/09/29 14:52:48 UTC
[30/41] hbase-site git commit: Published site at
fdbaa4c3f0253700d12b6bd61cacf29ca7fbc19f.
http://git-wip-us.apache.org/repos/asf/hbase-site/blob/83fd16f2/devapidocs/src-html/org/apache/hadoop/hbase/client/AsyncRequestFutureImpl.SingleServerRequestRunnable.html
----------------------------------------------------------------------
diff --git a/devapidocs/src-html/org/apache/hadoop/hbase/client/AsyncRequestFutureImpl.SingleServerRequestRunnable.html b/devapidocs/src-html/org/apache/hadoop/hbase/client/AsyncRequestFutureImpl.SingleServerRequestRunnable.html
index ad93f10..32aa215 100644
--- a/devapidocs/src-html/org/apache/hadoop/hbase/client/AsyncRequestFutureImpl.SingleServerRequestRunnable.html
+++ b/devapidocs/src-html/org/apache/hadoop/hbase/client/AsyncRequestFutureImpl.SingleServerRequestRunnable.html
@@ -834,448 +834,450 @@
<span class="sourceLineNo">826</span> byte[] regionName = regionEntry.getKey();<a name="line.826"></a>
<span class="sourceLineNo">827</span><a name="line.827"></a>
<span class="sourceLineNo">828</span> Throwable regionException = responses.getExceptions().get(regionName);<a name="line.828"></a>
-<span class="sourceLineNo">829</span> cleanServerCache(server, regionException);<a name="line.829"></a>
-<span class="sourceLineNo">830</span><a name="line.830"></a>
-<span class="sourceLineNo">831</span> Map<Integer, Object> regionResults =<a name="line.831"></a>
-<span class="sourceLineNo">832</span> results.containsKey(regionName) ? results.get(regionName).result : Collections.emptyMap();<a name="line.832"></a>
-<span class="sourceLineNo">833</span> boolean regionFailureRegistered = false;<a name="line.833"></a>
-<span class="sourceLineNo">834</span> for (Action sentAction : regionEntry.getValue()) {<a name="line.834"></a>
-<span class="sourceLineNo">835</span> Object result = regionResults.get(sentAction.getOriginalIndex());<a name="line.835"></a>
-<span class="sourceLineNo">836</span> if (result == null) {<a name="line.836"></a>
-<span class="sourceLineNo">837</span> if (regionException == null) {<a name="line.837"></a>
-<span class="sourceLineNo">838</span> LOG.error("Server sent us neither results nor exceptions for "<a name="line.838"></a>
-<span class="sourceLineNo">839</span> + Bytes.toStringBinary(regionName)<a name="line.839"></a>
-<span class="sourceLineNo">840</span> + ", numAttempt:" + numAttempt);<a name="line.840"></a>
-<span class="sourceLineNo">841</span> regionException = new RuntimeException("Invalid response");<a name="line.841"></a>
-<span class="sourceLineNo">842</span> }<a name="line.842"></a>
-<span class="sourceLineNo">843</span> // If the row operation encounters the region-lever error, the exception of action may be<a name="line.843"></a>
-<span class="sourceLineNo">844</span> // null.<a name="line.844"></a>
-<span class="sourceLineNo">845</span> result = regionException;<a name="line.845"></a>
-<span class="sourceLineNo">846</span> }<a name="line.846"></a>
-<span class="sourceLineNo">847</span> // Failure: retry if it's make sense else update the errors lists<a name="line.847"></a>
-<span class="sourceLineNo">848</span> if (result instanceof Throwable) {<a name="line.848"></a>
-<span class="sourceLineNo">849</span> Throwable actionException = (Throwable) result;<a name="line.849"></a>
-<span class="sourceLineNo">850</span> Row row = sentAction.getAction();<a name="line.850"></a>
-<span class="sourceLineNo">851</span> lastException = regionException != null ? regionException<a name="line.851"></a>
-<span class="sourceLineNo">852</span> : ClientExceptionsUtil.findException(actionException);<a name="line.852"></a>
-<span class="sourceLineNo">853</span> // Register corresponding failures once per server/once per region.<a name="line.853"></a>
-<span class="sourceLineNo">854</span> if (!regionFailureRegistered) {<a name="line.854"></a>
-<span class="sourceLineNo">855</span> regionFailureRegistered = true;<a name="line.855"></a>
-<span class="sourceLineNo">856</span> updateCachedLocations(server, regionName, row.getRow(), actionException);<a name="line.856"></a>
-<span class="sourceLineNo">857</span> }<a name="line.857"></a>
-<span class="sourceLineNo">858</span> if (retry == null) {<a name="line.858"></a>
-<span class="sourceLineNo">859</span> errorsByServer.reportServerError(server);<a name="line.859"></a>
-<span class="sourceLineNo">860</span> // We determine canRetry only once for all calls, after reporting server failure.<a name="line.860"></a>
-<span class="sourceLineNo">861</span> retry = errorsByServer.canTryMore(numAttempt) ?<a name="line.861"></a>
-<span class="sourceLineNo">862</span> Retry.YES : Retry.NO_RETRIES_EXHAUSTED;<a name="line.862"></a>
-<span class="sourceLineNo">863</span> }<a name="line.863"></a>
-<span class="sourceLineNo">864</span> ++failureCount;<a name="line.864"></a>
-<span class="sourceLineNo">865</span> switch (manageError(sentAction.getOriginalIndex(), row, retry, actionException,<a name="line.865"></a>
-<span class="sourceLineNo">866</span> server)) {<a name="line.866"></a>
-<span class="sourceLineNo">867</span> case YES:<a name="line.867"></a>
-<span class="sourceLineNo">868</span> toReplay.add(sentAction);<a name="line.868"></a>
-<span class="sourceLineNo">869</span> break;<a name="line.869"></a>
-<span class="sourceLineNo">870</span> case NO_OTHER_SUCCEEDED:<a name="line.870"></a>
-<span class="sourceLineNo">871</span> ++stopped;<a name="line.871"></a>
-<span class="sourceLineNo">872</span> break;<a name="line.872"></a>
-<span class="sourceLineNo">873</span> default:<a name="line.873"></a>
-<span class="sourceLineNo">874</span> ++failed;<a name="line.874"></a>
-<span class="sourceLineNo">875</span> break;<a name="line.875"></a>
-<span class="sourceLineNo">876</span> }<a name="line.876"></a>
-<span class="sourceLineNo">877</span> } else {<a name="line.877"></a>
-<span class="sourceLineNo">878</span> invokeCallBack(regionName, sentAction.getAction().getRow(), (CResult) result);<a name="line.878"></a>
-<span class="sourceLineNo">879</span> setResult(sentAction, result);<a name="line.879"></a>
-<span class="sourceLineNo">880</span> }<a name="line.880"></a>
-<span class="sourceLineNo">881</span> }<a name="line.881"></a>
-<span class="sourceLineNo">882</span> }<a name="line.882"></a>
-<span class="sourceLineNo">883</span> if (toReplay.isEmpty()) {<a name="line.883"></a>
-<span class="sourceLineNo">884</span> logNoResubmit(server, numAttempt, failureCount, lastException, failed, stopped);<a name="line.884"></a>
-<span class="sourceLineNo">885</span> } else {<a name="line.885"></a>
-<span class="sourceLineNo">886</span> resubmit(server, toReplay, numAttempt, failureCount, lastException);<a name="line.886"></a>
-<span class="sourceLineNo">887</span> }<a name="line.887"></a>
-<span class="sourceLineNo">888</span> }<a name="line.888"></a>
-<span class="sourceLineNo">889</span><a name="line.889"></a>
-<span class="sourceLineNo">890</span> private void updateCachedLocations(ServerName server, byte[] regionName, byte[] row,<a name="line.890"></a>
-<span class="sourceLineNo">891</span> Throwable rowException) {<a name="line.891"></a>
-<span class="sourceLineNo">892</span> if (tableName == null) {<a name="line.892"></a>
-<span class="sourceLineNo">893</span> return;<a name="line.893"></a>
-<span class="sourceLineNo">894</span> }<a name="line.894"></a>
-<span class="sourceLineNo">895</span> try {<a name="line.895"></a>
-<span class="sourceLineNo">896</span> asyncProcess.connection<a name="line.896"></a>
-<span class="sourceLineNo">897</span> .updateCachedLocations(tableName, regionName, row, rowException, server);<a name="line.897"></a>
-<span class="sourceLineNo">898</span> } catch (Throwable ex) {<a name="line.898"></a>
-<span class="sourceLineNo">899</span> // That should never happen, but if it did, we want to make sure<a name="line.899"></a>
-<span class="sourceLineNo">900</span> // we still process errors<a name="line.900"></a>
-<span class="sourceLineNo">901</span> LOG.error("Couldn't update cached region locations: " + ex);<a name="line.901"></a>
-<span class="sourceLineNo">902</span> }<a name="line.902"></a>
-<span class="sourceLineNo">903</span> }<a name="line.903"></a>
-<span class="sourceLineNo">904</span><a name="line.904"></a>
-<span class="sourceLineNo">905</span> private void invokeCallBack(byte[] regionName, byte[] row, CResult result) {<a name="line.905"></a>
-<span class="sourceLineNo">906</span> if (callback != null) {<a name="line.906"></a>
-<span class="sourceLineNo">907</span> try {<a name="line.907"></a>
-<span class="sourceLineNo">908</span> //noinspection unchecked<a name="line.908"></a>
-<span class="sourceLineNo">909</span> // TODO: would callback expect a replica region name if it gets one?<a name="line.909"></a>
-<span class="sourceLineNo">910</span> this.callback.update(regionName, row, result);<a name="line.910"></a>
-<span class="sourceLineNo">911</span> } catch (Throwable t) {<a name="line.911"></a>
-<span class="sourceLineNo">912</span> LOG.error("User callback threw an exception for "<a name="line.912"></a>
-<span class="sourceLineNo">913</span> + Bytes.toStringBinary(regionName) + ", ignoring", t);<a name="line.913"></a>
-<span class="sourceLineNo">914</span> }<a name="line.914"></a>
-<span class="sourceLineNo">915</span> }<a name="line.915"></a>
-<span class="sourceLineNo">916</span> }<a name="line.916"></a>
-<span class="sourceLineNo">917</span><a name="line.917"></a>
-<span class="sourceLineNo">918</span> private void cleanServerCache(ServerName server, Throwable regionException) {<a name="line.918"></a>
-<span class="sourceLineNo">919</span> if (tableName == null && ClientExceptionsUtil.isMetaClearingException(regionException)) {<a name="line.919"></a>
-<span class="sourceLineNo">920</span> // For multi-actions, we don't have a table name, but we want to make sure to clear the<a name="line.920"></a>
-<span class="sourceLineNo">921</span> // cache in case there were location-related exceptions. We don't to clear the cache<a name="line.921"></a>
-<span class="sourceLineNo">922</span> // for every possible exception that comes through, however.<a name="line.922"></a>
-<span class="sourceLineNo">923</span> asyncProcess.connection.clearCaches(server);<a name="line.923"></a>
-<span class="sourceLineNo">924</span> }<a name="line.924"></a>
-<span class="sourceLineNo">925</span> }<a name="line.925"></a>
-<span class="sourceLineNo">926</span><a name="line.926"></a>
-<span class="sourceLineNo">927</span> @VisibleForTesting<a name="line.927"></a>
-<span class="sourceLineNo">928</span> protected void updateStats(ServerName server, Map<byte[], MultiResponse.RegionResult> results) {<a name="line.928"></a>
-<span class="sourceLineNo">929</span> boolean metrics = asyncProcess.connection.getConnectionMetrics() != null;<a name="line.929"></a>
-<span class="sourceLineNo">930</span> boolean stats = asyncProcess.connection.getStatisticsTracker() != null;<a name="line.930"></a>
-<span class="sourceLineNo">931</span> if (!stats && !metrics) {<a name="line.931"></a>
-<span class="sourceLineNo">932</span> return;<a name="line.932"></a>
-<span class="sourceLineNo">933</span> }<a name="line.933"></a>
-<span class="sourceLineNo">934</span> for (Map.Entry<byte[], MultiResponse.RegionResult> regionStats : results.entrySet()) {<a name="line.934"></a>
-<span class="sourceLineNo">935</span> byte[] regionName = regionStats.getKey();<a name="line.935"></a>
-<span class="sourceLineNo">936</span> ClientProtos.RegionLoadStats stat = regionStats.getValue().getStat();<a name="line.936"></a>
-<span class="sourceLineNo">937</span> if (stat == null) {<a name="line.937"></a>
-<span class="sourceLineNo">938</span> LOG.error("No ClientProtos.RegionLoadStats found for server=" + server<a name="line.938"></a>
-<span class="sourceLineNo">939</span> + ", region=" + Bytes.toStringBinary(regionName));<a name="line.939"></a>
-<span class="sourceLineNo">940</span> continue;<a name="line.940"></a>
-<span class="sourceLineNo">941</span> }<a name="line.941"></a>
-<span class="sourceLineNo">942</span> RegionLoadStats regionLoadstats = ProtobufUtil.createRegionLoadStats(stat);<a name="line.942"></a>
-<span class="sourceLineNo">943</span> ResultStatsUtil.updateStats(asyncProcess.connection.getStatisticsTracker(), server,<a name="line.943"></a>
-<span class="sourceLineNo">944</span> regionName, regionLoadstats);<a name="line.944"></a>
-<span class="sourceLineNo">945</span> ResultStatsUtil.updateStats(asyncProcess.connection.getConnectionMetrics(),<a name="line.945"></a>
-<span class="sourceLineNo">946</span> server, regionName, regionLoadstats);<a name="line.946"></a>
-<span class="sourceLineNo">947</span> }<a name="line.947"></a>
-<span class="sourceLineNo">948</span> }<a name="line.948"></a>
-<span class="sourceLineNo">949</span><a name="line.949"></a>
-<span class="sourceLineNo">950</span><a name="line.950"></a>
-<span class="sourceLineNo">951</span> private String createLog(int numAttempt, int failureCount, int replaySize, ServerName sn,<a name="line.951"></a>
-<span class="sourceLineNo">952</span> Throwable error, long backOffTime, boolean willRetry, String startTime,<a name="line.952"></a>
-<span class="sourceLineNo">953</span> int failed, int stopped) {<a name="line.953"></a>
-<span class="sourceLineNo">954</span> StringBuilder sb = new StringBuilder();<a name="line.954"></a>
-<span class="sourceLineNo">955</span> sb.append("id=").append(asyncProcess.id).append(", table=").append(tableName).<a name="line.955"></a>
-<span class="sourceLineNo">956</span> append(", attempt=").append(numAttempt).append("/").append(asyncProcess.numTries).<a name="line.956"></a>
-<span class="sourceLineNo">957</span> append(", ");<a name="line.957"></a>
-<span class="sourceLineNo">958</span><a name="line.958"></a>
-<span class="sourceLineNo">959</span> if (failureCount > 0 || error != null){<a name="line.959"></a>
-<span class="sourceLineNo">960</span> sb.append("failureCount=").append(failureCount).append("ops").append(", last exception=").<a name="line.960"></a>
-<span class="sourceLineNo">961</span> append(error);<a name="line.961"></a>
-<span class="sourceLineNo">962</span> } else {<a name="line.962"></a>
-<span class="sourceLineNo">963</span> sb.append("succeeded");<a name="line.963"></a>
-<span class="sourceLineNo">964</span> }<a name="line.964"></a>
-<span class="sourceLineNo">965</span><a name="line.965"></a>
-<span class="sourceLineNo">966</span> sb.append(" on ").append(sn).append(", tracking started ").append(startTime);<a name="line.966"></a>
+<span class="sourceLineNo">829</span> if (regionException != null) {<a name="line.829"></a>
+<span class="sourceLineNo">830</span> cleanServerCache(server, regionException);<a name="line.830"></a>
+<span class="sourceLineNo">831</span> }<a name="line.831"></a>
+<span class="sourceLineNo">832</span><a name="line.832"></a>
+<span class="sourceLineNo">833</span> Map<Integer, Object> regionResults =<a name="line.833"></a>
+<span class="sourceLineNo">834</span> results.containsKey(regionName) ? results.get(regionName).result : Collections.emptyMap();<a name="line.834"></a>
+<span class="sourceLineNo">835</span> boolean regionFailureRegistered = false;<a name="line.835"></a>
+<span class="sourceLineNo">836</span> for (Action sentAction : regionEntry.getValue()) {<a name="line.836"></a>
+<span class="sourceLineNo">837</span> Object result = regionResults.get(sentAction.getOriginalIndex());<a name="line.837"></a>
+<span class="sourceLineNo">838</span> if (result == null) {<a name="line.838"></a>
+<span class="sourceLineNo">839</span> if (regionException == null) {<a name="line.839"></a>
+<span class="sourceLineNo">840</span> LOG.error("Server sent us neither results nor exceptions for "<a name="line.840"></a>
+<span class="sourceLineNo">841</span> + Bytes.toStringBinary(regionName)<a name="line.841"></a>
+<span class="sourceLineNo">842</span> + ", numAttempt:" + numAttempt);<a name="line.842"></a>
+<span class="sourceLineNo">843</span> regionException = new RuntimeException("Invalid response");<a name="line.843"></a>
+<span class="sourceLineNo">844</span> }<a name="line.844"></a>
+<span class="sourceLineNo">845</span> // If the row operation encounters the region-lever error, the exception of action may be<a name="line.845"></a>
+<span class="sourceLineNo">846</span> // null.<a name="line.846"></a>
+<span class="sourceLineNo">847</span> result = regionException;<a name="line.847"></a>
+<span class="sourceLineNo">848</span> }<a name="line.848"></a>
+<span class="sourceLineNo">849</span> // Failure: retry if it's make sense else update the errors lists<a name="line.849"></a>
+<span class="sourceLineNo">850</span> if (result instanceof Throwable) {<a name="line.850"></a>
+<span class="sourceLineNo">851</span> Throwable actionException = (Throwable) result;<a name="line.851"></a>
+<span class="sourceLineNo">852</span> Row row = sentAction.getAction();<a name="line.852"></a>
+<span class="sourceLineNo">853</span> lastException = regionException != null ? regionException<a name="line.853"></a>
+<span class="sourceLineNo">854</span> : ClientExceptionsUtil.findException(actionException);<a name="line.854"></a>
+<span class="sourceLineNo">855</span> // Register corresponding failures once per server/once per region.<a name="line.855"></a>
+<span class="sourceLineNo">856</span> if (!regionFailureRegistered) {<a name="line.856"></a>
+<span class="sourceLineNo">857</span> regionFailureRegistered = true;<a name="line.857"></a>
+<span class="sourceLineNo">858</span> updateCachedLocations(server, regionName, row.getRow(), actionException);<a name="line.858"></a>
+<span class="sourceLineNo">859</span> }<a name="line.859"></a>
+<span class="sourceLineNo">860</span> if (retry == null) {<a name="line.860"></a>
+<span class="sourceLineNo">861</span> errorsByServer.reportServerError(server);<a name="line.861"></a>
+<span class="sourceLineNo">862</span> // We determine canRetry only once for all calls, after reporting server failure.<a name="line.862"></a>
+<span class="sourceLineNo">863</span> retry = errorsByServer.canTryMore(numAttempt) ?<a name="line.863"></a>
+<span class="sourceLineNo">864</span> Retry.YES : Retry.NO_RETRIES_EXHAUSTED;<a name="line.864"></a>
+<span class="sourceLineNo">865</span> }<a name="line.865"></a>
+<span class="sourceLineNo">866</span> ++failureCount;<a name="line.866"></a>
+<span class="sourceLineNo">867</span> switch (manageError(sentAction.getOriginalIndex(), row, retry, actionException,<a name="line.867"></a>
+<span class="sourceLineNo">868</span> server)) {<a name="line.868"></a>
+<span class="sourceLineNo">869</span> case YES:<a name="line.869"></a>
+<span class="sourceLineNo">870</span> toReplay.add(sentAction);<a name="line.870"></a>
+<span class="sourceLineNo">871</span> break;<a name="line.871"></a>
+<span class="sourceLineNo">872</span> case NO_OTHER_SUCCEEDED:<a name="line.872"></a>
+<span class="sourceLineNo">873</span> ++stopped;<a name="line.873"></a>
+<span class="sourceLineNo">874</span> break;<a name="line.874"></a>
+<span class="sourceLineNo">875</span> default:<a name="line.875"></a>
+<span class="sourceLineNo">876</span> ++failed;<a name="line.876"></a>
+<span class="sourceLineNo">877</span> break;<a name="line.877"></a>
+<span class="sourceLineNo">878</span> }<a name="line.878"></a>
+<span class="sourceLineNo">879</span> } else {<a name="line.879"></a>
+<span class="sourceLineNo">880</span> invokeCallBack(regionName, sentAction.getAction().getRow(), (CResult) result);<a name="line.880"></a>
+<span class="sourceLineNo">881</span> setResult(sentAction, result);<a name="line.881"></a>
+<span class="sourceLineNo">882</span> }<a name="line.882"></a>
+<span class="sourceLineNo">883</span> }<a name="line.883"></a>
+<span class="sourceLineNo">884</span> }<a name="line.884"></a>
+<span class="sourceLineNo">885</span> if (toReplay.isEmpty()) {<a name="line.885"></a>
+<span class="sourceLineNo">886</span> logNoResubmit(server, numAttempt, failureCount, lastException, failed, stopped);<a name="line.886"></a>
+<span class="sourceLineNo">887</span> } else {<a name="line.887"></a>
+<span class="sourceLineNo">888</span> resubmit(server, toReplay, numAttempt, failureCount, lastException);<a name="line.888"></a>
+<span class="sourceLineNo">889</span> }<a name="line.889"></a>
+<span class="sourceLineNo">890</span> }<a name="line.890"></a>
+<span class="sourceLineNo">891</span><a name="line.891"></a>
+<span class="sourceLineNo">892</span> private void updateCachedLocations(ServerName server, byte[] regionName, byte[] row,<a name="line.892"></a>
+<span class="sourceLineNo">893</span> Throwable rowException) {<a name="line.893"></a>
+<span class="sourceLineNo">894</span> if (tableName == null) {<a name="line.894"></a>
+<span class="sourceLineNo">895</span> return;<a name="line.895"></a>
+<span class="sourceLineNo">896</span> }<a name="line.896"></a>
+<span class="sourceLineNo">897</span> try {<a name="line.897"></a>
+<span class="sourceLineNo">898</span> asyncProcess.connection<a name="line.898"></a>
+<span class="sourceLineNo">899</span> .updateCachedLocations(tableName, regionName, row, rowException, server);<a name="line.899"></a>
+<span class="sourceLineNo">900</span> } catch (Throwable ex) {<a name="line.900"></a>
+<span class="sourceLineNo">901</span> // That should never happen, but if it did, we want to make sure<a name="line.901"></a>
+<span class="sourceLineNo">902</span> // we still process errors<a name="line.902"></a>
+<span class="sourceLineNo">903</span> LOG.error("Couldn't update cached region locations: " + ex);<a name="line.903"></a>
+<span class="sourceLineNo">904</span> }<a name="line.904"></a>
+<span class="sourceLineNo">905</span> }<a name="line.905"></a>
+<span class="sourceLineNo">906</span><a name="line.906"></a>
+<span class="sourceLineNo">907</span> private void invokeCallBack(byte[] regionName, byte[] row, CResult result) {<a name="line.907"></a>
+<span class="sourceLineNo">908</span> if (callback != null) {<a name="line.908"></a>
+<span class="sourceLineNo">909</span> try {<a name="line.909"></a>
+<span class="sourceLineNo">910</span> //noinspection unchecked<a name="line.910"></a>
+<span class="sourceLineNo">911</span> // TODO: would callback expect a replica region name if it gets one?<a name="line.911"></a>
+<span class="sourceLineNo">912</span> this.callback.update(regionName, row, result);<a name="line.912"></a>
+<span class="sourceLineNo">913</span> } catch (Throwable t) {<a name="line.913"></a>
+<span class="sourceLineNo">914</span> LOG.error("User callback threw an exception for "<a name="line.914"></a>
+<span class="sourceLineNo">915</span> + Bytes.toStringBinary(regionName) + ", ignoring", t);<a name="line.915"></a>
+<span class="sourceLineNo">916</span> }<a name="line.916"></a>
+<span class="sourceLineNo">917</span> }<a name="line.917"></a>
+<span class="sourceLineNo">918</span> }<a name="line.918"></a>
+<span class="sourceLineNo">919</span><a name="line.919"></a>
+<span class="sourceLineNo">920</span> private void cleanServerCache(ServerName server, Throwable regionException) {<a name="line.920"></a>
+<span class="sourceLineNo">921</span> if (tableName == null && ClientExceptionsUtil.isMetaClearingException(regionException)) {<a name="line.921"></a>
+<span class="sourceLineNo">922</span> // For multi-actions, we don't have a table name, but we want to make sure to clear the<a name="line.922"></a>
+<span class="sourceLineNo">923</span> // cache in case there were location-related exceptions. We don't to clear the cache<a name="line.923"></a>
+<span class="sourceLineNo">924</span> // for every possible exception that comes through, however.<a name="line.924"></a>
+<span class="sourceLineNo">925</span> asyncProcess.connection.clearCaches(server);<a name="line.925"></a>
+<span class="sourceLineNo">926</span> }<a name="line.926"></a>
+<span class="sourceLineNo">927</span> }<a name="line.927"></a>
+<span class="sourceLineNo">928</span><a name="line.928"></a>
+<span class="sourceLineNo">929</span> @VisibleForTesting<a name="line.929"></a>
+<span class="sourceLineNo">930</span> protected void updateStats(ServerName server, Map<byte[], MultiResponse.RegionResult> results) {<a name="line.930"></a>
+<span class="sourceLineNo">931</span> boolean metrics = asyncProcess.connection.getConnectionMetrics() != null;<a name="line.931"></a>
+<span class="sourceLineNo">932</span> boolean stats = asyncProcess.connection.getStatisticsTracker() != null;<a name="line.932"></a>
+<span class="sourceLineNo">933</span> if (!stats && !metrics) {<a name="line.933"></a>
+<span class="sourceLineNo">934</span> return;<a name="line.934"></a>
+<span class="sourceLineNo">935</span> }<a name="line.935"></a>
+<span class="sourceLineNo">936</span> for (Map.Entry<byte[], MultiResponse.RegionResult> regionStats : results.entrySet()) {<a name="line.936"></a>
+<span class="sourceLineNo">937</span> byte[] regionName = regionStats.getKey();<a name="line.937"></a>
+<span class="sourceLineNo">938</span> ClientProtos.RegionLoadStats stat = regionStats.getValue().getStat();<a name="line.938"></a>
+<span class="sourceLineNo">939</span> if (stat == null) {<a name="line.939"></a>
+<span class="sourceLineNo">940</span> LOG.error("No ClientProtos.RegionLoadStats found for server=" + server<a name="line.940"></a>
+<span class="sourceLineNo">941</span> + ", region=" + Bytes.toStringBinary(regionName));<a name="line.941"></a>
+<span class="sourceLineNo">942</span> continue;<a name="line.942"></a>
+<span class="sourceLineNo">943</span> }<a name="line.943"></a>
+<span class="sourceLineNo">944</span> RegionLoadStats regionLoadstats = ProtobufUtil.createRegionLoadStats(stat);<a name="line.944"></a>
+<span class="sourceLineNo">945</span> ResultStatsUtil.updateStats(asyncProcess.connection.getStatisticsTracker(), server,<a name="line.945"></a>
+<span class="sourceLineNo">946</span> regionName, regionLoadstats);<a name="line.946"></a>
+<span class="sourceLineNo">947</span> ResultStatsUtil.updateStats(asyncProcess.connection.getConnectionMetrics(),<a name="line.947"></a>
+<span class="sourceLineNo">948</span> server, regionName, regionLoadstats);<a name="line.948"></a>
+<span class="sourceLineNo">949</span> }<a name="line.949"></a>
+<span class="sourceLineNo">950</span> }<a name="line.950"></a>
+<span class="sourceLineNo">951</span><a name="line.951"></a>
+<span class="sourceLineNo">952</span><a name="line.952"></a>
+<span class="sourceLineNo">953</span> private String createLog(int numAttempt, int failureCount, int replaySize, ServerName sn,<a name="line.953"></a>
+<span class="sourceLineNo">954</span> Throwable error, long backOffTime, boolean willRetry, String startTime,<a name="line.954"></a>
+<span class="sourceLineNo">955</span> int failed, int stopped) {<a name="line.955"></a>
+<span class="sourceLineNo">956</span> StringBuilder sb = new StringBuilder();<a name="line.956"></a>
+<span class="sourceLineNo">957</span> sb.append("id=").append(asyncProcess.id).append(", table=").append(tableName).<a name="line.957"></a>
+<span class="sourceLineNo">958</span> append(", attempt=").append(numAttempt).append("/").append(asyncProcess.numTries).<a name="line.958"></a>
+<span class="sourceLineNo">959</span> append(", ");<a name="line.959"></a>
+<span class="sourceLineNo">960</span><a name="line.960"></a>
+<span class="sourceLineNo">961</span> if (failureCount > 0 || error != null){<a name="line.961"></a>
+<span class="sourceLineNo">962</span> sb.append("failureCount=").append(failureCount).append("ops").append(", last exception=").<a name="line.962"></a>
+<span class="sourceLineNo">963</span> append(error);<a name="line.963"></a>
+<span class="sourceLineNo">964</span> } else {<a name="line.964"></a>
+<span class="sourceLineNo">965</span> sb.append("succeeded");<a name="line.965"></a>
+<span class="sourceLineNo">966</span> }<a name="line.966"></a>
<span class="sourceLineNo">967</span><a name="line.967"></a>
-<span class="sourceLineNo">968</span> if (willRetry) {<a name="line.968"></a>
-<span class="sourceLineNo">969</span> sb.append(", retrying after=").append(backOffTime).append("ms").<a name="line.969"></a>
-<span class="sourceLineNo">970</span> append(", operationsToReplay=").append(replaySize);<a name="line.970"></a>
-<span class="sourceLineNo">971</span> } else if (failureCount > 0) {<a name="line.971"></a>
-<span class="sourceLineNo">972</span> if (stopped > 0) {<a name="line.972"></a>
-<span class="sourceLineNo">973</span> sb.append("; NOT retrying, stopped=").append(stopped).<a name="line.973"></a>
-<span class="sourceLineNo">974</span> append(" because successful operation on other replica");<a name="line.974"></a>
-<span class="sourceLineNo">975</span> }<a name="line.975"></a>
-<span class="sourceLineNo">976</span> if (failed > 0) {<a name="line.976"></a>
-<span class="sourceLineNo">977</span> sb.append("; NOT retrying, failed=").append(failed).append(" -- final attempt!");<a name="line.977"></a>
-<span class="sourceLineNo">978</span> }<a name="line.978"></a>
-<span class="sourceLineNo">979</span> }<a name="line.979"></a>
-<span class="sourceLineNo">980</span><a name="line.980"></a>
-<span class="sourceLineNo">981</span> return sb.toString();<a name="line.981"></a>
-<span class="sourceLineNo">982</span> }<a name="line.982"></a>
-<span class="sourceLineNo">983</span><a name="line.983"></a>
-<span class="sourceLineNo">984</span> /**<a name="line.984"></a>
-<span class="sourceLineNo">985</span> * Sets the non-error result from a particular action.<a name="line.985"></a>
-<span class="sourceLineNo">986</span> * @param action Action (request) that the server responded to.<a name="line.986"></a>
-<span class="sourceLineNo">987</span> * @param result The result.<a name="line.987"></a>
-<span class="sourceLineNo">988</span> */<a name="line.988"></a>
-<span class="sourceLineNo">989</span> private void setResult(Action action, Object result) {<a name="line.989"></a>
-<span class="sourceLineNo">990</span> if (result == null) {<a name="line.990"></a>
-<span class="sourceLineNo">991</span> throw new RuntimeException("Result cannot be null");<a name="line.991"></a>
-<span class="sourceLineNo">992</span> }<a name="line.992"></a>
-<span class="sourceLineNo">993</span> ReplicaResultState state = null;<a name="line.993"></a>
-<span class="sourceLineNo">994</span> boolean isStale = !RegionReplicaUtil.isDefaultReplica(action.getReplicaId());<a name="line.994"></a>
-<span class="sourceLineNo">995</span> int index = action.getOriginalIndex();<a name="line.995"></a>
-<span class="sourceLineNo">996</span> if (results == null) {<a name="line.996"></a>
-<span class="sourceLineNo">997</span> decActionCounter(index);<a name="line.997"></a>
-<span class="sourceLineNo">998</span> return; // Simple case, no replica requests.<a name="line.998"></a>
-<span class="sourceLineNo">999</span> }<a name="line.999"></a>
-<span class="sourceLineNo">1000</span> state = trySetResultSimple(index, action.getAction(), false, result, null, isStale);<a name="line.1000"></a>
-<span class="sourceLineNo">1001</span> if (state == null) {<a name="line.1001"></a>
-<span class="sourceLineNo">1002</span> return; // Simple case, no replica requests.<a name="line.1002"></a>
-<span class="sourceLineNo">1003</span> }<a name="line.1003"></a>
-<span class="sourceLineNo">1004</span> // At this point we know that state is set to replica tracking class.<a name="line.1004"></a>
-<span class="sourceLineNo">1005</span> // It could be that someone else is also looking at it; however, we know there can<a name="line.1005"></a>
-<span class="sourceLineNo">1006</span> // only be one state object, and only one thread can set callCount to 0. Other threads<a name="line.1006"></a>
-<span class="sourceLineNo">1007</span> // will either see state with callCount 0 after locking it; or will not see state at all<a name="line.1007"></a>
-<span class="sourceLineNo">1008</span> // we will replace it with the result.<a name="line.1008"></a>
-<span class="sourceLineNo">1009</span> synchronized (state) {<a name="line.1009"></a>
-<span class="sourceLineNo">1010</span> if (state.callCount == 0) {<a name="line.1010"></a>
-<span class="sourceLineNo">1011</span> return; // someone already set the result<a name="line.1011"></a>
-<span class="sourceLineNo">1012</span> }<a name="line.1012"></a>
-<span class="sourceLineNo">1013</span> state.callCount = 0;<a name="line.1013"></a>
-<span class="sourceLineNo">1014</span> }<a name="line.1014"></a>
-<span class="sourceLineNo">1015</span> synchronized (replicaResultLock) {<a name="line.1015"></a>
-<span class="sourceLineNo">1016</span> if (results[index] != state) {<a name="line.1016"></a>
-<span class="sourceLineNo">1017</span> throw new AssertionError("We set the callCount but someone else replaced the result");<a name="line.1017"></a>
-<span class="sourceLineNo">1018</span> }<a name="line.1018"></a>
-<span class="sourceLineNo">1019</span> updateResult(index, result);<a name="line.1019"></a>
-<span class="sourceLineNo">1020</span> }<a name="line.1020"></a>
-<span class="sourceLineNo">1021</span><a name="line.1021"></a>
-<span class="sourceLineNo">1022</span> decActionCounter(index);<a name="line.1022"></a>
-<span class="sourceLineNo">1023</span> }<a name="line.1023"></a>
-<span class="sourceLineNo">1024</span><a name="line.1024"></a>
-<span class="sourceLineNo">1025</span> /**<a name="line.1025"></a>
-<span class="sourceLineNo">1026</span> * Sets the error from a particular action.<a name="line.1026"></a>
-<span class="sourceLineNo">1027</span> * @param index Original action index.<a name="line.1027"></a>
-<span class="sourceLineNo">1028</span> * @param row Original request.<a name="line.1028"></a>
-<span class="sourceLineNo">1029</span> * @param throwable The resulting error.<a name="line.1029"></a>
-<span class="sourceLineNo">1030</span> * @param server The source server.<a name="line.1030"></a>
-<span class="sourceLineNo">1031</span> */<a name="line.1031"></a>
-<span class="sourceLineNo">1032</span> private void setError(int index, Row row, Throwable throwable, ServerName server) {<a name="line.1032"></a>
-<span class="sourceLineNo">1033</span> ReplicaResultState state = null;<a name="line.1033"></a>
-<span class="sourceLineNo">1034</span> if (results == null) {<a name="line.1034"></a>
-<span class="sourceLineNo">1035</span> // Note that we currently cannot have replica requests with null results. So it shouldn't<a name="line.1035"></a>
-<span class="sourceLineNo">1036</span> // happen that multiple replica calls will call dAC for same actions with results == null.<a name="line.1036"></a>
-<span class="sourceLineNo">1037</span> // Only one call per action should be present in this case.<a name="line.1037"></a>
-<span class="sourceLineNo">1038</span> errors.add(throwable, row, server);<a name="line.1038"></a>
-<span class="sourceLineNo">1039</span> decActionCounter(index);<a name="line.1039"></a>
-<span class="sourceLineNo">1040</span> return; // Simple case, no replica requests.<a name="line.1040"></a>
-<span class="sourceLineNo">1041</span> }<a name="line.1041"></a>
-<span class="sourceLineNo">1042</span> state = trySetResultSimple(index, row, true, throwable, server, false);<a name="line.1042"></a>
-<span class="sourceLineNo">1043</span> if (state == null) {<a name="line.1043"></a>
-<span class="sourceLineNo">1044</span> return; // Simple case, no replica requests.<a name="line.1044"></a>
-<span class="sourceLineNo">1045</span> }<a name="line.1045"></a>
-<span class="sourceLineNo">1046</span> BatchErrors target = null; // Error will be added to final errors, or temp replica errors.<a name="line.1046"></a>
-<span class="sourceLineNo">1047</span> boolean isActionDone = false;<a name="line.1047"></a>
-<span class="sourceLineNo">1048</span> synchronized (state) {<a name="line.1048"></a>
-<span class="sourceLineNo">1049</span> switch (state.callCount) {<a name="line.1049"></a>
-<span class="sourceLineNo">1050</span> case 0: return; // someone already set the result<a name="line.1050"></a>
-<span class="sourceLineNo">1051</span> case 1: { // All calls failed, we are the last error.<a name="line.1051"></a>
-<span class="sourceLineNo">1052</span> target = errors;<a name="line.1052"></a>
-<span class="sourceLineNo">1053</span> isActionDone = true;<a name="line.1053"></a>
-<span class="sourceLineNo">1054</span> break;<a name="line.1054"></a>
-<span class="sourceLineNo">1055</span> }<a name="line.1055"></a>
-<span class="sourceLineNo">1056</span> default: {<a name="line.1056"></a>
-<span class="sourceLineNo">1057</span> assert state.callCount > 1;<a name="line.1057"></a>
-<span class="sourceLineNo">1058</span> if (state.replicaErrors == null) {<a name="line.1058"></a>
-<span class="sourceLineNo">1059</span> state.replicaErrors = new BatchErrors();<a name="line.1059"></a>
-<span class="sourceLineNo">1060</span> }<a name="line.1060"></a>
-<span class="sourceLineNo">1061</span> target = state.replicaErrors;<a name="line.1061"></a>
-<span class="sourceLineNo">1062</span> break;<a name="line.1062"></a>
-<span class="sourceLineNo">1063</span> }<a name="line.1063"></a>
-<span class="sourceLineNo">1064</span> }<a name="line.1064"></a>
-<span class="sourceLineNo">1065</span> --state.callCount;<a name="line.1065"></a>
-<span class="sourceLineNo">1066</span> }<a name="line.1066"></a>
-<span class="sourceLineNo">1067</span> target.add(throwable, row, server);<a name="line.1067"></a>
-<span class="sourceLineNo">1068</span> if (isActionDone) {<a name="line.1068"></a>
-<span class="sourceLineNo">1069</span> if (state.replicaErrors != null) { // last call, no need to lock<a name="line.1069"></a>
-<span class="sourceLineNo">1070</span> errors.merge(state.replicaErrors);<a name="line.1070"></a>
-<span class="sourceLineNo">1071</span> }<a name="line.1071"></a>
-<span class="sourceLineNo">1072</span> // See setResult for explanations.<a name="line.1072"></a>
-<span class="sourceLineNo">1073</span> synchronized (replicaResultLock) {<a name="line.1073"></a>
-<span class="sourceLineNo">1074</span> if (results[index] != state) {<a name="line.1074"></a>
-<span class="sourceLineNo">1075</span> throw new AssertionError("We set the callCount but someone else replaced the result");<a name="line.1075"></a>
-<span class="sourceLineNo">1076</span> }<a name="line.1076"></a>
-<span class="sourceLineNo">1077</span> updateResult(index, throwable);<a name="line.1077"></a>
-<span class="sourceLineNo">1078</span> }<a name="line.1078"></a>
-<span class="sourceLineNo">1079</span> decActionCounter(index);<a name="line.1079"></a>
-<span class="sourceLineNo">1080</span> }<a name="line.1080"></a>
-<span class="sourceLineNo">1081</span> }<a name="line.1081"></a>
-<span class="sourceLineNo">1082</span><a name="line.1082"></a>
-<span class="sourceLineNo">1083</span> /**<a name="line.1083"></a>
-<span class="sourceLineNo">1084</span> * Checks if the action is complete; used on error to prevent needless retries.<a name="line.1084"></a>
-<span class="sourceLineNo">1085</span> * Does not synchronize, assuming element index/field accesses are atomic.<a name="line.1085"></a>
-<span class="sourceLineNo">1086</span> * This is an opportunistic optimization check, doesn't have to be strict.<a name="line.1086"></a>
-<span class="sourceLineNo">1087</span> * @param index Original action index.<a name="line.1087"></a>
-<span class="sourceLineNo">1088</span> * @param row Original request.<a name="line.1088"></a>
-<span class="sourceLineNo">1089</span> */<a name="line.1089"></a>
-<span class="sourceLineNo">1090</span> private boolean isActionComplete(int index, Row row) {<a name="line.1090"></a>
-<span class="sourceLineNo">1091</span> if (!AsyncProcess.isReplicaGet(row)) return false;<a name="line.1091"></a>
-<span class="sourceLineNo">1092</span> Object resObj = results[index];<a name="line.1092"></a>
-<span class="sourceLineNo">1093</span> return (resObj != null) && (!(resObj instanceof ReplicaResultState)<a name="line.1093"></a>
-<span class="sourceLineNo">1094</span> || ((ReplicaResultState)resObj).callCount == 0);<a name="line.1094"></a>
-<span class="sourceLineNo">1095</span> }<a name="line.1095"></a>
-<span class="sourceLineNo">1096</span><a name="line.1096"></a>
-<span class="sourceLineNo">1097</span> /**<a name="line.1097"></a>
-<span class="sourceLineNo">1098</span> * Tries to set the result or error for a particular action as if there were no replica calls.<a name="line.1098"></a>
-<span class="sourceLineNo">1099</span> * @return null if successful; replica state if there were in fact replica calls.<a name="line.1099"></a>
-<span class="sourceLineNo">1100</span> */<a name="line.1100"></a>
-<span class="sourceLineNo">1101</span> private ReplicaResultState trySetResultSimple(int index, Row row, boolean isError,<a name="line.1101"></a>
-<span class="sourceLineNo">1102</span> Object result, ServerName server, boolean isFromReplica) {<a name="line.1102"></a>
-<span class="sourceLineNo">1103</span> Object resObj = null;<a name="line.1103"></a>
-<span class="sourceLineNo">1104</span> if (!AsyncProcess.isReplicaGet(row)) {<a name="line.1104"></a>
-<span class="sourceLineNo">1105</span> if (isFromReplica) {<a name="line.1105"></a>
-<span class="sourceLineNo">1106</span> throw new AssertionError("Unexpected stale result for " + row);<a name="line.1106"></a>
-<span class="sourceLineNo">1107</span> }<a name="line.1107"></a>
-<span class="sourceLineNo">1108</span> updateResult(index, result);<a name="line.1108"></a>
-<span class="sourceLineNo">1109</span> } else {<a name="line.1109"></a>
-<span class="sourceLineNo">1110</span> synchronized (replicaResultLock) {<a name="line.1110"></a>
-<span class="sourceLineNo">1111</span> resObj = results[index];<a name="line.1111"></a>
-<span class="sourceLineNo">1112</span> if (resObj == null) {<a name="line.1112"></a>
-<span class="sourceLineNo">1113</span> if (isFromReplica) {<a name="line.1113"></a>
-<span class="sourceLineNo">1114</span> throw new AssertionError("Unexpected stale result for " + row);<a name="line.1114"></a>
-<span class="sourceLineNo">1115</span> }<a name="line.1115"></a>
-<span class="sourceLineNo">1116</span> updateResult(index, result);<a name="line.1116"></a>
-<span class="sourceLineNo">1117</span> }<a name="line.1117"></a>
-<span class="sourceLineNo">1118</span> }<a name="line.1118"></a>
-<span class="sourceLineNo">1119</span> }<a name="line.1119"></a>
-<span class="sourceLineNo">1120</span><a name="line.1120"></a>
-<span class="sourceLineNo">1121</span> ReplicaResultState rrs =<a name="line.1121"></a>
-<span class="sourceLineNo">1122</span> (resObj instanceof ReplicaResultState) ? (ReplicaResultState)resObj : null;<a name="line.1122"></a>
-<span class="sourceLineNo">1123</span> if (rrs == null && isError) {<a name="line.1123"></a>
-<span class="sourceLineNo">1124</span> // The resObj is not replica state (null or already set).<a name="line.1124"></a>
-<span class="sourceLineNo">1125</span> errors.add((Throwable)result, row, server);<a name="line.1125"></a>
-<span class="sourceLineNo">1126</span> }<a name="line.1126"></a>
-<span class="sourceLineNo">1127</span><a name="line.1127"></a>
-<span class="sourceLineNo">1128</span> if (resObj == null) {<a name="line.1128"></a>
-<span class="sourceLineNo">1129</span> // resObj is null - no replica calls were made.<a name="line.1129"></a>
-<span class="sourceLineNo">1130</span> decActionCounter(index);<a name="line.1130"></a>
-<span class="sourceLineNo">1131</span> return null;<a name="line.1131"></a>
-<span class="sourceLineNo">1132</span> }<a name="line.1132"></a>
-<span class="sourceLineNo">1133</span> return rrs;<a name="line.1133"></a>
-<span class="sourceLineNo">1134</span> }<a name="line.1134"></a>
-<span class="sourceLineNo">1135</span><a name="line.1135"></a>
-<span class="sourceLineNo">1136</span> private void decActionCounter(int index) {<a name="line.1136"></a>
-<span class="sourceLineNo">1137</span> long actionsRemaining = actionsInProgress.decrementAndGet();<a name="line.1137"></a>
-<span class="sourceLineNo">1138</span> if (actionsRemaining < 0) {<a name="line.1138"></a>
-<span class="sourceLineNo">1139</span> String error = buildDetailedErrorMsg("Incorrect actions in progress", index);<a name="line.1139"></a>
-<span class="sourceLineNo">1140</span> throw new AssertionError(error);<a name="line.1140"></a>
-<span class="sourceLineNo">1141</span> } else if (actionsRemaining == 0) {<a name="line.1141"></a>
-<span class="sourceLineNo">1142</span> synchronized (actionsInProgress) {<a name="line.1142"></a>
-<span class="sourceLineNo">1143</span> actionsInProgress.notifyAll();<a name="line.1143"></a>
-<span class="sourceLineNo">1144</span> }<a name="line.1144"></a>
-<span class="sourceLineNo">1145</span> }<a name="line.1145"></a>
-<span class="sourceLineNo">1146</span> }<a name="line.1146"></a>
-<span class="sourceLineNo">1147</span><a name="line.1147"></a>
-<span class="sourceLineNo">1148</span> private String buildDetailedErrorMsg(String string, int index) {<a name="line.1148"></a>
-<span class="sourceLineNo">1149</span> StringBuilder error = new StringBuilder(128);<a name="line.1149"></a>
-<span class="sourceLineNo">1150</span> error.append(string).append("; called for ").append(index).append(", actionsInProgress ")<a name="line.1150"></a>
-<span class="sourceLineNo">1151</span> .append(actionsInProgress.get()).append("; replica gets: ");<a name="line.1151"></a>
-<span class="sourceLineNo">1152</span> if (replicaGetIndices != null) {<a name="line.1152"></a>
-<span class="sourceLineNo">1153</span> for (int i = 0; i < replicaGetIndices.length; ++i) {<a name="line.1153"></a>
-<span class="sourceLineNo">1154</span> error.append(replicaGetIndices[i]).append(", ");<a name="line.1154"></a>
-<span class="sourceLineNo">1155</span> }<a name="line.1155"></a>
-<span class="sourceLineNo">1156</span> } else {<a name="line.1156"></a>
-<span class="sourceLineNo">1157</span> error.append(hasAnyReplicaGets ? "all" : "none");<a name="line.1157"></a>
-<span class="sourceLineNo">1158</span> }<a name="line.1158"></a>
-<span class="sourceLineNo">1159</span> error.append("; results ");<a name="line.1159"></a>
-<span class="sourceLineNo">1160</span> if (results != null) {<a name="line.1160"></a>
-<span class="sourceLineNo">1161</span> for (int i = 0; i < results.length; ++i) {<a name="line.1161"></a>
-<span class="sourceLineNo">1162</span> Object o = results[i];<a name="line.1162"></a>
-<span class="sourceLineNo">1163</span> error.append(((o == null) ? "null" : o.toString())).append(", ");<a name="line.1163"></a>
-<span class="sourceLineNo">1164</span> }<a name="line.1164"></a>
-<span class="sourceLineNo">1165</span> }<a name="line.1165"></a>
-<span class="sourceLineNo">1166</span> return error.toString();<a name="line.1166"></a>
-<span class="sourceLineNo">1167</span> }<a name="line.1167"></a>
-<span class="sourceLineNo">1168</span><a name="line.1168"></a>
-<span class="sourceLineNo">1169</span> @Override<a name="line.1169"></a>
-<span class="sourceLineNo">1170</span> public void waitUntilDone() throws InterruptedIOException {<a name="line.1170"></a>
-<span class="sourceLineNo">1171</span> try {<a name="line.1171"></a>
-<span class="sourceLineNo">1172</span> waitUntilDone(Long.MAX_VALUE);<a name="line.1172"></a>
-<span class="sourceLineNo">1173</span> } catch (InterruptedException iex) {<a name="line.1173"></a>
-<span class="sourceLineNo">1174</span> throw new InterruptedIOException(iex.getMessage());<a name="line.1174"></a>
-<span class="sourceLineNo">1175</span> } finally {<a name="line.1175"></a>
-<span class="sourceLineNo">1176</span> if (callsInProgress != null) {<a name="line.1176"></a>
-<span class="sourceLineNo">1177</span> for (CancellableRegionServerCallable clb : callsInProgress) {<a name="line.1177"></a>
-<span class="sourceLineNo">1178</span> clb.cancel();<a name="line.1178"></a>
-<span class="sourceLineNo">1179</span> }<a name="line.1179"></a>
-<span class="sourceLineNo">1180</span> }<a name="line.1180"></a>
-<span class="sourceLineNo">1181</span> }<a name="line.1181"></a>
-<span class="sourceLineNo">1182</span> }<a name="line.1182"></a>
-<span class="sourceLineNo">1183</span><a name="line.1183"></a>
-<span class="sourceLineNo">1184</span> private boolean waitUntilDone(long cutoff) throws InterruptedException {<a name="line.1184"></a>
-<span class="sourceLineNo">1185</span> boolean hasWait = cutoff != Long.MAX_VALUE;<a name="line.1185"></a>
-<span class="sourceLineNo">1186</span> long lastLog = EnvironmentEdgeManager.currentTime();<a name="line.1186"></a>
-<span class="sourceLineNo">1187</span> long currentInProgress;<a name="line.1187"></a>
-<span class="sourceLineNo">1188</span> while (0 != (currentInProgress = actionsInProgress.get())) {<a name="line.1188"></a>
-<span class="sourceLineNo">1189</span> long now = EnvironmentEdgeManager.currentTime();<a name="line.1189"></a>
-<span class="sourceLineNo">1190</span> if (hasWait && (now * 1000L) > cutoff) {<a name="line.1190"></a>
-<span class="sourceLineNo">1191</span> return false;<a name="line.1191"></a>
-<span class="sourceLineNo">1192</span> }<a name="line.1192"></a>
-<span class="sourceLineNo">1193</span> if (!hasWait) { // Only log if wait is infinite.<a name="line.1193"></a>
-<span class="sourceLineNo">1194</span> if (now > lastLog + 10000) {<a name="line.1194"></a>
-<span class="sourceLineNo">1195</span> lastLog = now;<a name="line.1195"></a>
-<span class="sourceLineNo">1196</span> LOG.info("#" + asyncProcess.id + ", waiting for " + currentInProgress<a name="line.1196"></a>
-<span class="sourceLineNo">1197</span> + " actions to finish on table: " + tableName);<a name="line.1197"></a>
-<span class="sourceLineNo">1198</span> }<a name="line.1198"></a>
-<span class="sourceLineNo">1199</span> }<a name="line.1199"></a>
-<span class="sourceLineNo">1200</span> synchronized (actionsInProgress) {<a name="line.1200"></a>
-<span class="sourceLineNo">1201</span> if (actionsInProgress.get() == 0) break;<a name="line.1201"></a>
-<span class="sourceLineNo">1202</span> if (!hasWait) {<a name="line.1202"></a>
-<span class="sourceLineNo">1203</span> actionsInProgress.wait(10);<a name="line.1203"></a>
-<span class="sourceLineNo">1204</span> } else {<a name="line.1204"></a>
-<span class="sourceLineNo">1205</span> long waitMicroSecond = Math.min(100000L, (cutoff - now * 1000L));<a name="line.1205"></a>
-<span class="sourceLineNo">1206</span> TimeUnit.MICROSECONDS.timedWait(actionsInProgress, waitMicroSecond);<a name="line.1206"></a>
-<span class="sourceLineNo">1207</span> }<a name="line.1207"></a>
-<span class="sourceLineNo">1208</span> }<a name="line.1208"></a>
-<span class="sourceLineNo">1209</span> }<a name="line.1209"></a>
-<span class="sourceLineNo">1210</span> return true;<a name="line.1210"></a>
-<span class="sourceLineNo">1211</span> }<a name="line.1211"></a>
-<span class="sourceLineNo">1212</span><a name="line.1212"></a>
-<span class="sourceLineNo">1213</span> @Override<a name="line.1213"></a>
-<span class="sourceLineNo">1214</span> public boolean hasError() {<a name="line.1214"></a>
-<span class="sourceLineNo">1215</span> return errors.hasErrors();<a name="line.1215"></a>
-<span class="sourceLineNo">1216</span> }<a name="line.1216"></a>
-<span class="sourceLineNo">1217</span><a name="line.1217"></a>
-<span class="sourceLineNo">1218</span> @Override<a name="line.1218"></a>
-<span class="sourceLineNo">1219</span> public List<? extends Row> getFailedOperations() {<a name="line.1219"></a>
-<span class="sourceLineNo">1220</span> return errors.actions;<a name="line.1220"></a>
-<span class="sourceLineNo">1221</span> }<a name="line.1221"></a>
-<span class="sourceLineNo">1222</span><a name="line.1222"></a>
-<span class="sourceLineNo">1223</span> @Override<a name="line.1223"></a>
-<span class="sourceLineNo">1224</span> public RetriesExhaustedWithDetailsException getErrors() {<a name="line.1224"></a>
-<span class="sourceLineNo">1225</span> return errors.makeException(asyncProcess.logBatchErrorDetails);<a name="line.1225"></a>
-<span class="sourceLineNo">1226</span> }<a name="line.1226"></a>
-<span class="sourceLineNo">1227</span><a name="line.1227"></a>
-<span class="sourceLineNo">1228</span> @Override<a name="line.1228"></a>
-<span class="sourceLineNo">1229</span> public Object[] getResults() throws InterruptedIOException {<a name="line.1229"></a>
-<span class="sourceLineNo">1230</span> waitUntilDone();<a name="line.1230"></a>
-<span class="sourceLineNo">1231</span> return results;<a name="line.1231"></a>
-<span class="sourceLineNo">1232</span> }<a name="line.1232"></a>
-<span class="sourceLineNo">1233</span><a name="line.1233"></a>
-<span class="sourceLineNo">1234</span> /**<a name="line.1234"></a>
-<span class="sourceLineNo">1235</span> * Creates the server error tracker to use inside process.<a name="line.1235"></a>
-<span class="sourceLineNo">1236</span> * Currently, to preserve the main assumption about current retries, and to work well with<a name="line.1236"></a>
-<span class="sourceLineNo">1237</span> * the retry-limit-based calculation, the calculation is local per Process object.<a name="line.1237"></a>
-<span class="sourceLineNo">1238</span> * We may benefit from connection-wide tracking of server errors.<a name="line.1238"></a>
-<span class="sourceLineNo">1239</span> * @return ServerErrorTracker to use, null if there is no ServerErrorTracker on this connection<a name="line.1239"></a>
-<span class="sourceLineNo">1240</span> */<a name="line.1240"></a>
-<span class="sourceLineNo">1241</span> private ConnectionImplementation.ServerErrorTracker createServerErrorTracker() {<a name="line.1241"></a>
-<span class="sourceLineNo">1242</span> return new ConnectionImplementation.ServerErrorTracker(<a name="line.1242"></a>
-<span class="sourceLineNo">1243</span> asyncProcess.serverTrackerTimeout, asyncProcess.numTries);<a name="line.1243"></a>
-<span class="sourceLineNo">1244</span> }<a name="line.1244"></a>
-<span class="sourceLineNo">1245</span><a name="line.1245"></a>
-<span class="sourceLineNo">1246</span> /**<a name="line.1246"></a>
-<span class="sourceLineNo">1247</span> * Create a callable. Isolated to be easily overridden in the tests.<a name="line.1247"></a>
-<span class="sourceLineNo">1248</span> */<a name="line.1248"></a>
-<span class="sourceLineNo">1249</span> private MultiServerCallable createCallable(final ServerName server, TableName tableName,<a name="line.1249"></a>
-<span class="sourceLineNo">1250</span> final MultiAction multi) {<a name="line.1250"></a>
-<span class="sourceLineNo">1251</span> return new MultiServerCallable(asyncProcess.connection, tableName, server,<a name="line.1251"></a>
-<span class="sourceLineNo">1252</span> multi, asyncProcess.rpcFactory.newController(), rpcTimeout, tracker, multi.getPriority());<a name="line.1252"></a>
-<span class="sourceLineNo">1253</span> }<a name="line.1253"></a>
-<span class="sourceLineNo">1254</span><a name="line.1254"></a>
-<span class="sourceLineNo">1255</span> private void updateResult(int index, Object result) {<a name="line.1255"></a>
-<span class="sourceLineNo">1256</span> Object current = results[index];<a name="line.1256"></a>
-<span class="sourceLineNo">1257</span> if (current != null) {<a name="line.1257"></a>
-<span class="sourceLineNo">1258</span> if (LOG.isDebugEnabled()) {<a name="line.1258"></a>
-<span class="sourceLineNo">1259</span> LOG.debug("The result is assigned repeatedly! current:" + current<a name="line.1259"></a>
-<span class="sourceLineNo">1260</span> + ", new:" + result);<a name="line.1260"></a>
-<span class="sourceLineNo">1261</span> }<a name="line.1261"></a>
-<span class="sourceLineNo">1262</span> }<a name="line.1262"></a>
-<span class="sourceLineNo">1263</span> results[index] = result;<a name="line.1263"></a>
-<span class="sourceLineNo">1264</span> }<a name="line.1264"></a>
-<span class="sourceLineNo">1265</span><a name="line.1265"></a>
-<span class="sourceLineNo">1266</span> @VisibleForTesting<a name="line.1266"></a>
-<span class="sourceLineNo">1267</span> long getNumberOfActionsInProgress() {<a name="line.1267"></a>
-<span class="sourceLineNo">1268</span> return actionsInProgress.get();<a name="line.1268"></a>
-<span class="sourceLineNo">1269</span> }<a name="line.1269"></a>
-<span class="sourceLineNo">1270</span>}<a name="line.1270"></a>
+<span class="sourceLineNo">968</span> sb.append(" on ").append(sn).append(", tracking started ").append(startTime);<a name="line.968"></a>
+<span class="sourceLineNo">969</span><a name="line.969"></a>
+<span class="sourceLineNo">970</span> if (willRetry) {<a name="line.970"></a>
+<span class="sourceLineNo">971</span> sb.append(", retrying after=").append(backOffTime).append("ms").<a name="line.971"></a>
+<span class="sourceLineNo">972</span> append(", operationsToReplay=").append(replaySize);<a name="line.972"></a>
+<span class="sourceLineNo">973</span> } else if (failureCount > 0) {<a name="line.973"></a>
+<span class="sourceLineNo">974</span> if (stopped > 0) {<a name="line.974"></a>
+<span class="sourceLineNo">975</span> sb.append("; NOT retrying, stopped=").append(stopped).<a name="line.975"></a>
+<span class="sourceLineNo">976</span> append(" because successful operation on other replica");<a name="line.976"></a>
+<span class="sourceLineNo">977</span> }<a name="line.977"></a>
+<span class="sourceLineNo">978</span> if (failed > 0) {<a name="line.978"></a>
+<span class="sourceLineNo">979</span> sb.append("; NOT retrying, failed=").append(failed).append(" -- final attempt!");<a name="line.979"></a>
+<span class="sourceLineNo">980</span> }<a name="line.980"></a>
+<span class="sourceLineNo">981</span> }<a name="line.981"></a>
+<span class="sourceLineNo">982</span><a name="line.982"></a>
+<span class="sourceLineNo">983</span> return sb.toString();<a name="line.983"></a>
+<span class="sourceLineNo">984</span> }<a name="line.984"></a>
+<span class="sourceLineNo">985</span><a name="line.985"></a>
+<span class="sourceLineNo">986</span> /**<a name="line.986"></a>
+<span class="sourceLineNo">987</span> * Sets the non-error result from a particular action.<a name="line.987"></a>
+<span class="sourceLineNo">988</span> * @param action Action (request) that the server responded to.<a name="line.988"></a>
+<span class="sourceLineNo">989</span> * @param result The result.<a name="line.989"></a>
+<span class="sourceLineNo">990</span> */<a name="line.990"></a>
+<span class="sourceLineNo">991</span> private void setResult(Action action, Object result) {<a name="line.991"></a>
+<span class="sourceLineNo">992</span> if (result == null) {<a name="line.992"></a>
+<span class="sourceLineNo">993</span> throw new RuntimeException("Result cannot be null");<a name="line.993"></a>
+<span class="sourceLineNo">994</span> }<a name="line.994"></a>
+<span class="sourceLineNo">995</span> ReplicaResultState state = null;<a name="line.995"></a>
+<span class="sourceLineNo">996</span> boolean isStale = !RegionReplicaUtil.isDefaultReplica(action.getReplicaId());<a name="line.996"></a>
+<span class="sourceLineNo">997</span> int index = action.getOriginalIndex();<a name="line.997"></a>
+<span class="sourceLineNo">998</span> if (results == null) {<a name="line.998"></a>
+<span class="sourceLineNo">999</span> decActionCounter(index);<a name="line.999"></a>
+<span class="sourceLineNo">1000</span> return; // Simple case, no replica requests.<a name="line.1000"></a>
+<span class="sourceLineNo">1001</span> }<a name="line.1001"></a>
+<span class="sourceLineNo">1002</span> state = trySetResultSimple(index, action.getAction(), false, result, null, isStale);<a name="line.1002"></a>
+<span class="sourceLineNo">1003</span> if (state == null) {<a name="line.1003"></a>
+<span class="sourceLineNo">1004</span> return; // Simple case, no replica requests.<a name="line.1004"></a>
+<span class="sourceLineNo">1005</span> }<a name="line.1005"></a>
+<span class="sourceLineNo">1006</span> // At this point we know that state is set to replica tracking class.<a name="line.1006"></a>
+<span class="sourceLineNo">1007</span> // It could be that someone else is also looking at it; however, we know there can<a name="line.1007"></a>
+<span class="sourceLineNo">1008</span> // only be one state object, and only one thread can set callCount to 0. Other threads<a name="line.1008"></a>
+<span class="sourceLineNo">1009</span> // will either see state with callCount 0 after locking it; or will not see state at all<a name="line.1009"></a>
+<span class="sourceLineNo">1010</span> // we will replace it with the result.<a name="line.1010"></a>
+<span class="sourceLineNo">1011</span> synchronized (state) {<a name="line.1011"></a>
+<span class="sourceLineNo">1012</span> if (state.callCount == 0) {<a name="line.1012"></a>
+<span class="sourceLineNo">1013</span> return; // someone already set the result<a name="line.1013"></a>
+<span class="sourceLineNo">1014</span> }<a name="line.1014"></a>
+<span class="sourceLineNo">1015</span> state.callCount = 0;<a name="line.1015"></a>
+<span class="sourceLineNo">1016</span> }<a name="line.1016"></a>
+<span class="sourceLineNo">1017</span> synchronized (replicaResultLock) {<a name="line.1017"></a>
+<span class="sourceLineNo">1018</span> if (results[index] != state) {<a name="line.1018"></a>
+<span class="sourceLineNo">1019</span> throw new AssertionError("We set the callCount but someone else replaced the result");<a name="line.1019"></a>
+<span class="sourceLineNo">1020</span> }<a name="line.1020"></a>
+<span class="sourceLineNo">1021</span> updateResult(index, result);<a name="line.1021"></a>
+<span class="sourceLineNo">1022</span> }<a name="line.1022"></a>
+<span class="sourceLineNo">1023</span><a name="line.1023"></a>
+<span class="sourceLineNo">1024</span> decActionCounter(index);<a name="line.1024"></a>
+<span class="sourceLineNo">1025</span> }<a name="line.1025"></a>
+<span class="sourceLineNo">1026</span><a name="line.1026"></a>
+<span class="sourceLineNo">1027</span> /**<a name="line.1027"></a>
+<span class="sourceLineNo">1028</span> * Sets the error from a particular action.<a name="line.1028"></a>
+<span class="sourceLineNo">1029</span> * @param index Original action index.<a name="line.1029"></a>
+<span class="sourceLineNo">1030</span> * @param row Original request.<a name="line.1030"></a>
+<span class="sourceLineNo">1031</span> * @param throwable The resulting error.<a name="line.1031"></a>
+<span class="sourceLineNo">1032</span> * @param server The source server.<a name="line.1032"></a>
+<span class="sourceLineNo">1033</span> */<a name="line.1033"></a>
+<span class="sourceLineNo">1034</span> private void setError(int index, Row row, Throwable throwable, ServerName server) {<a name="line.1034"></a>
+<span class="sourceLineNo">1035</span> ReplicaResultState state = null;<a name="line.1035"></a>
+<span class="sourceLineNo">1036</span> if (results == null) {<a name="line.1036"></a>
+<span class="sourceLineNo">1037</span> // Note that we currently cannot have replica requests with null results. So it shouldn't<a name="line.1037"></a>
+<span class="sourceLineNo">1038</span> // happen that multiple replica calls will call dAC for same actions with results == null.<a name="line.1038"></a>
+<span class="sourceLineNo">1039</span> // Only one call per action should be present in this case.<a name="line.1039"></a>
+<span class="sourceLineNo">1040</span> errors.add(throwable, row, server);<a name="line.1040"></a>
+<span class="sourceLineNo">1041</span> decActionCounter(index);<a name="line.1041"></a>
+<span class="sourceLineNo">1042</span> return; // Simple case, no replica requests.<a name="line.1042"></a>
+<span class="sourceLineNo">1043</span> }<a name="line.1043"></a>
+<span class="sourceLineNo">1044</span> state = trySetResultSimple(index, row, true, throwable, server, false);<a name="line.1044"></a>
+<span class="sourceLineNo">1045</span> if (state == null) {<a name="line.1045"></a>
+<span class="sourceLineNo">1046</span> return; // Simple case, no replica requests.<a name="line.1046"></a>
+<span class="sourceLineNo">1047</span> }<a name="line.1047"></a>
+<span class="sourceLineNo">1048</span> BatchErrors target = null; // Error will be added to final errors, or temp replica errors.<a name="line.1048"></a>
+<span class="sourceLineNo">1049</span> boolean isActionDone = false;<a name="line.1049"></a>
+<span class="sourceLineNo">1050</span> synchronized (state) {<a name="line.1050"></a>
+<span class="sourceLineNo">1051</span> switch (state.callCount) {<a name="line.1051"></a>
+<span class="sourceLineNo">1052</span> case 0: return; // someone already set the result<a name="line.1052"></a>
+<span class="sourceLineNo">1053</span> case 1: { // All calls failed, we are the last error.<a name="line.1053"></a>
+<span class="sourceLineNo">1054</span> target = errors;<a name="line.1054"></a>
+<span class="sourceLineNo">1055</span> isActionDone = true;<a name="line.1055"></a>
+<span class="sourceLineNo">1056</span> break;<a name="line.1056"></a>
+<span class="sourceLineNo">1057</span> }<a name="line.1057"></a>
+<span class="sourceLineNo">1058</span> default: {<a name="line.1058"></a>
+<span class="sourceLineNo">1059</span> assert state.callCount > 1;<a name="line.1059"></a>
+<span class="sourceLineNo">1060</span> if (state.replicaErrors == null) {<a name="line.1060"></a>
+<span class="sourceLineNo">1061</span> state.replicaErrors = new BatchErrors();<a name="line.1061"></a>
+<span class="sourceLineNo">1062</span> }<a name="line.1062"></a>
+<span class="sourceLineNo">1063</span> target = state.replicaErrors;<a name="line.1063"></a>
+<span class="sourceLineNo">1064</span> break;<a name="line.1064"></a>
+<span class="sourceLineNo">1065</span> }<a name="line.1065"></a>
+<span class="sourceLineNo">1066</span> }<a name="line.1066"></a>
+<span class="sourceLineNo">1067</span> --state.callCount;<a name="line.1067"></a>
+<span class="sourceLineNo">1068</span> }<a name="line.1068"></a>
+<span class="sourceLineNo">1069</span> target.add(throwable, row, server);<a name="line.1069"></a>
+<span class="sourceLineNo">1070</span> if (isActionDone) {<a name="line.1070"></a>
+<span class="sourceLineNo">1071</span> if (state.replicaErrors != null) { // last call, no need to lock<a name="line.1071"></a>
+<span class="sourceLineNo">1072</span> errors.merge(state.replicaErrors);<a name="line.1072"></a>
+<span class="sourceLineNo">1073</span> }<a name="line.1073"></a>
+<span class="sourceLineNo">1074</span> // See setResult for explanations.<a name="line.1074"></a>
+<span class="sourceLineNo">1075</span> synchronized (replicaResultLock) {<a name="line.1075"></a>
+<span class="sourceLineNo">1076</span> if (results[index] != state) {<a name="line.1076"></a>
+<span class="sourceLineNo">1077</span> throw new AssertionError("We set the callCount but someone else replaced the result");<a name="line.1077"></a>
+<span class="sourceLineNo">1078</span> }<a name="line.1078"></a>
+<span class="sourceLineNo">1079</span> updateResult(index, throwable);<a name="line.1079"></a>
+<span class="sourceLineNo">1080</span> }<a name="line.1080"></a>
+<span class="sourceLineNo">1081</span> decActionCounter(index);<a name="line.1081"></a>
+<span class="sourceLineNo">1082</span> }<a name="line.1082"></a>
+<span class="sourceLineNo">1083</span> }<a name="line.1083"></a>
+<span class="sourceLineNo">1084</span><a name="line.1084"></a>
+<span class="sourceLineNo">1085</span> /**<a name="line.1085"></a>
+<span class="sourceLineNo">1086</span> * Checks if the action is complete; used on error to prevent needless retries.<a name="line.1086"></a>
+<span class="sourceLineNo">1087</span> * Does not synchronize, assuming element index/field accesses are atomic.<a name="line.1087"></a>
+<span class="sourceLineNo">1088</span> * This is an opportunistic optimization check, doesn't have to be strict.<a name="line.1088"></a>
+<span class="sourceLineNo">1089</span> * @param index Original action index.<a name="line.1089"></a>
+<span class="sourceLineNo">1090</span> * @param row Original request.<a name="line.1090"></a>
+<span class="sourceLineNo">1091</span> */<a name="line.1091"></a>
+<span class="sourceLineNo">1092</span> private boolean isActionComplete(int index, Row row) {<a name="line.1092"></a>
+<span class="sourceLineNo">1093</span> if (!AsyncProcess.isReplicaGet(row)) return false;<a name="line.1093"></a>
+<span class="sourceLineNo">1094</span> Object resObj = results[index];<a name="line.1094"></a>
+<span class="sourceLineNo">1095</span> return (resObj != null) && (!(resObj instanceof ReplicaResultState)<a name="line.1095"></a>
+<span class="sourceLineNo">1096</span> || ((ReplicaResultState)resObj).callCount == 0);<a name="line.1096"></a>
+<span class="sourceLineNo">1097</span> }<a name="line.1097"></a>
+<span class="sourceLineNo">1098</span><a name="line.1098"></a>
+<span class="sourceLineNo">1099</span> /**<a name="line.1099"></a>
+<span class="sourceLineNo">1100</span> * Tries to set the result or error for a particular action as if there were no replica calls.<a name="line.1100"></a>
+<span class="sourceLineNo">1101</span> * @return null if successful; replica state if there were in fact replica calls.<a name="line.1101"></a>
+<span class="sourceLineNo">1102</span> */<a name="line.1102"></a>
+<span class="sourceLineNo">1103</span> private ReplicaResultState trySetResultSimple(int index, Row row, boolean isError,<a name="line.1103"></a>
+<span class="sourceLineNo">1104</span> Object result, ServerName server, boolean isFromReplica) {<a name="line.1104"></a>
+<span class="sourceLineNo">1105</span> Object resObj = null;<a name="line.1105"></a>
+<span class="sourceLineNo">1106</span> if (!AsyncProcess.isReplicaGet(row)) {<a name="line.1106"></a>
+<span class="sourceLineNo">1107</span> if (isFromReplica) {<a name="line.1107"></a>
+<span class="sourceLineNo">1108</span> throw new AssertionError("Unexpected stale result for " + row);<a name="line.1108"></a>
+<span class="sourceLineNo">1109</span> }<a name="line.1109"></a>
+<span class="sourceLineNo">1110</span> updateResult(index, result);<a name="line.1110"></a>
+<span class="sourceLineNo">1111</span> } else {<a name="line.1111"></a>
+<span class="sourceLineNo">1112</span> synchronized (replicaResultLock) {<a name="line.1112"></a>
+<span class="sourceLineNo">1113</span> resObj = results[index];<a name="line.1113"></a>
+<span class="sourceLineNo">1114</span> if (resObj == null) {<a name="line.1114"></a>
+<span class="sourceLineNo">1115</span> if (isFromReplica) {<a name="line.1115"></a>
+<span class="sourceLineNo">1116</span> throw new AssertionError("Unexpected stale result for " + row);<a name="line.1116"></a>
+<span class="sourceLineNo">1117</span> }<a name="line.1117"></a>
+<span class="sourceLineNo">1118</span> updateResult(index, result);<a name="line.1118"></a>
+<span class="sourceLineNo">1119</span> }<a name="line.1119"></a>
+<span class="sourceLineNo">1120</span> }<a name="line.1120"></a>
+<span class="sourceLineNo">1121</span> }<a name="line.1121"></a>
+<span class="sourceLineNo">1122</span><a name="line.1122"></a>
+<span class="sourceLineNo">1123</span> ReplicaResultState rrs =<a name="line.1123"></a>
+<span class="sourceLineNo">1124</span> (resObj instanceof ReplicaResultState) ? (ReplicaResultState)resObj : null;<a name="line.1124"></a>
+<span class="sourceLineNo">1125</span> if (rrs == null && isError) {<a name="line.1125"></a>
+<span class="sourceLineNo">1126</span> // The resObj is not replica state (null or already set).<a name="line.1126"></a>
+<span class="sourceLineNo">1127</span> errors.add((Throwable)result, row, server);<a name="line.1127"></a>
+<span class="sourceLineNo">1128</span> }<a name="line.1128"></a>
+<span class="sourceLineNo">1129</span><a name="line.1129"></a>
+<span class="sourceLineNo">1130</span> if (resObj == null) {<a name="line.1130"></a>
+<span class="sourceLineNo">1131</span> // resObj is null - no replica calls were made.<a name="line.1131"></a>
+<span class="sourceLineNo">1132</span> decActionCounter(index);<a name="line.1132"></a>
+<span class="sourceLineNo">1133</span> return null;<a name="line.1133"></a>
+<span class="sourceLineNo">1134</span> }<a name="line.1134"></a>
+<span class="sourceLineNo">1135</span> return rrs;<a name="line.1135"></a>
+<span class="sourceLineNo">1136</span> }<a name="line.1136"></a>
+<span class="sourceLineNo">1137</span><a name="line.1137"></a>
+<span class="sourceLineNo">1138</span> private void decActionCounter(int index) {<a name="line.1138"></a>
+<span class="sourceLineNo">1139</span> long actionsRemaining = actionsInProgress.decrementAndGet();<a name="line.1139"></a>
+<span class="sourceLineNo">1140</span> if (actionsRemaining < 0) {<a name="line.1140"></a>
+<span class="sourceLineNo">1141</span> String error = buildDetailedErrorMsg("Incorrect actions in progress", index);<a name="line.1141"></a>
+<span class="sourceLineNo">1142</span> throw new AssertionError(error);<a name="line.1142"></a>
+<span class="sourceLineNo">1143</span> } else if (actionsRemaining == 0) {<a name="line.1143"></a>
+<span class="sourceLineNo">1144</span> synchronized (actionsInProgress) {<a name="line.1144"></a>
+<span class="sourceLineNo">1145</span> actionsInProgress.notifyAll();<a name="line.1145"></a>
+<span class="sourceLineNo">1146</span> }<a name="line.1146"></a>
+<span class="sourceLineNo">1147</span> }<a name="line.1147"></a>
+<span class="sourceLineNo">1148</span> }<a name="line.1148"></a>
+<span class="sourceLineNo">1149</span><a name="line.1149"></a>
+<span class="sourceLineNo">1150</span> private String buildDetailedErrorMsg(String string, int index) {<a name="line.1150"></a>
+<span class="sourceLineNo">1151</span> StringBuilder error = new StringBuilder(128);<a name="line.1151"></a>
+<span class="sourceLineNo">1152</span> error.append(string).append("; called for ").append(index).append(", actionsInProgress ")<a name="line.1152"></a>
+<span class="sourceLineNo">1153</span> .append(actionsInProgress.get()).append("; replica gets: ");<a name="line.1153"></a>
+<span class="sourceLineNo">1154</span> if (replicaGetIndices != null) {<a name="line.1154"></a>
+<span class="sourceLineNo">1155</span> for (int i = 0; i < replicaGetIndices.length; ++i) {<a name="line.1155"></a>
+<span class="sourceLineNo">1156</span> error.append(replicaGetIndices[i]).append(", ");<a name="line.1156"></a>
+<span class="sourceLineNo">1157</span> }<a name="line.1157"></a>
+<span class="sourceLineNo">1158</span> } else {<a name="line.1158"></a>
+<span class="sourceLineNo">1159</span> error.append(hasAnyReplicaGets ? "all" : "none");<a name="line.1159"></a>
+<span class="sourceLineNo">1160</span> }<a name="line.1160"></a>
+<span class="sourceLineNo">1161</span> error.append("; results ");<a name="line.1161"></a>
+<span class="sourceLineNo">1162</span> if (results != null) {<a name="line.1162"></a>
+<span class="sourceLineNo">1163</span> for (int i = 0; i < results.length; ++i) {<a name="line.1163"></a>
+<span class="sourceLineNo">1164</span> Object o = results[i];<a name="line.1164"></a>
+<span class="sourceLineNo">1165</span> error.append(((o == null) ? "null" : o.toString())).append(", ");<a name="line.1165"></a>
+<span class="sourceLineNo">1166</span> }<a name="line.1166"></a>
+<span class="sourceLineNo">1167</span> }<a name="line.1167"></a>
+<span class="sourceLineNo">1168</span> return error.toString();<a name="line.1168"></a>
+<span class="sourceLineNo">1169</span> }<a name="line.1169"></a>
+<span class="sourceLineNo">1170</span><a name="line.1170"></a>
+<span class="sourceLineNo">1171</span> @Override<a name="line.1171"></a>
+<span class="sourceLineNo">1172</span> public void waitUntilDone() throws InterruptedIOException {<a name="line.1172"></a>
+<span class="sourceLineNo">1173</span> try {<a name="line.1173"></a>
+<span class="sourceLineNo">1174</span> waitUntilDone(Long.MAX_VALUE);<a name="line.1174"></a>
+<span class="sourceLineNo">1175</span> } catch (InterruptedException iex) {<a name="line.1175"></a>
+<span class="sourceLineNo">1176</span> throw new InterruptedIOException(iex.getMessage());<a name="line.1176"></a>
+<span class="sourceLineNo">1177</span> } finally {<a name="line.1177"></a>
+<span class="sourceLineNo">1178</span> if (callsInProgress != null) {<a name="line.1178"></a>
+<span class="sourceLineNo">1179</span> for (CancellableRegionServerCallable clb : callsInProgress) {<a name="line.1179"></a>
+<span class="sourceLineNo">1180</span> clb.cancel();<a name="line.1180"></a>
+<span class="sourceLineNo">1181</span> }<a name="line.1181"></a>
+<span class="sourceLineNo">1182</span> }<a name="line.1182"></a>
+<span class="sourceLineNo">1183</span> }<a name="line.1183"></a>
+<span class="sourceLineNo">1184</span> }<a name="line.1184"></a>
+<span class="sourceLineNo">1185</span><a name="line.1185"></a>
+<span class="sourceLineNo">1186</span> private boolean waitUntilDone(long cutoff) throws InterruptedException {<a name="line.1186"></a>
+<span class="sourceLineNo">1187</span> boolean hasWait = cutoff != Long.MAX_VALUE;<a name="line.1187"></a>
+<span class="sourceLineNo">1188</span> long lastLog = EnvironmentEdgeManager.currentTime();<a name="line.1188"></a>
+<span class="sourceLineNo">1189</span> long currentInProgress;<a name="line.1189"></a>
+<span class="sourceLineNo">1190</span> while (0 != (currentInProgress = actionsInProgress.get())) {<a name="line.1190"></a>
+<span class="sourceLineNo">1191</span> long now = EnvironmentEdgeManager.currentTime();<a name="line.1191"></a>
+<span class="sourceLineNo">1192</span> if (hasWait && (now * 1000L) > cutoff) {<a name="line.1192"></a>
+<span class="sourceLineNo">1193</span> return false;<a name="line.1193"></a>
+<span class="sourceLineNo">1194</span> }<a name="line.1194"></a>
+<span class="sourceLineNo">1195</span> if (!hasWait) { // Only log if wait is infinite.<a name="line.1195"></a>
+<span class="sourceLineNo">1196</span> if (now > lastLog + 10000) {<a name="line.1196"></a>
+<span class="sourceLineNo">1197</span> lastLog = now;<a name="line.1197"></a>
+<span class="sourceLineNo">1198</span> LOG.info("#" + asyncProcess.id + ", waiting for " + currentInProgress<a name="line.1198"></a>
+<span class="sourceLineNo">1199</span> + " actions to finish on table: " + tableName);<a name="line.1199"></a>
+<span class="sourceLineNo">1200</span> }<a name="line.1200"></a>
+<span class="sourceLineNo">1201</span> }<a name="line.1201"></a>
+<span class="sourceLineNo">1202</span> synchronized (actionsInProgress) {<a name="line.1202"></a>
+<span class="sourceLineNo">1203</span> if (actionsInProgress.get() == 0) break;<a name="line.1203"></a>
+<span class="sourceLineNo">1204</span> if (!hasWait) {<a name="line.1204"></a>
+<span class="sourceLineNo">1205</span> actionsInProgress.wait(10);<a name="line.1205"></a>
+<span class="sourceLineNo">1206</span> } else {<a name="line.1206"></a>
+<span class="sourceLineNo">1207</span> long waitMicroSecond = Math.min(100000L, (cutoff - now * 1000L));<a name="line.1207"></a>
+<span class="sourceLineNo">1208</span> TimeUnit.MICROSECONDS.timedWait(actionsInProgress, waitMicroSecond);<a name="line.1208"></a>
+<span class="sourceLineNo">1209</span> }<a name="line.1209"></a>
+<span class="sourceLineNo">1210</span> }<a name="line.1210"></a>
+<span class="sourceLineNo">1211</span> }<a name="line.1211"></a>
+<span class="sourceLineNo">1212</span> return true;<a name="line.1212"></a>
+<span class="sourceLineNo">1213</span> }<a name="line.1213"></a>
+<span class="sourceLineNo">1214</span><a name="line.1214"></a>
+<span class="sourceLineNo">1215</span> @Override<a name="line.1215"></a>
+<span class="sourceLineNo">1216</span> public boolean hasError() {<a name="line.1216"></a>
+<span class="sourceLineNo">1217</span> return errors.hasErrors();<a name="line.1217"></a>
+<span class="sourceLineNo">1218</span> }<a name="line.1218"></a>
+<span class="sourceLineNo">1219</span><a name="line.1219"></a>
+<span class="sourceLineNo">1220</span> @Override<a name="line.1220"></a>
+<span class="sourceLineNo">1221</span> public List<? extends Row> getFailedOperations() {<a name="line.1221"></a>
+<span class="sourceLineNo">1222</span> return errors.actions;<a name="line.1222"></a>
+<span class="sourceLineNo">1223</span> }<a name="line.1223"></a>
+<span class="sourceLineNo">1224</span><a name="line.1224"></a>
+<span class="sourceLineNo">1225</span> @Override<a name="line.1225"></a>
+<span class="sourceLineNo">1226</span> public RetriesExhaustedWithDetailsException getErrors() {<a name="line.1226"></a>
+<span class="sourceLineNo">1227</span> return errors.makeException(asyncProcess.logBatchErrorDetails);<a name="line.1227"></a>
+<span class="sourceLineNo">1228</span> }<a name="line.1228"></a>
+<span class="sourceLineNo">1229</span><a name="line.1229"></a>
+<span class="sourceLineNo">1230</span> @Override<a name="line.1230"></a>
+<span class="sourceLineNo">1231</span> public Object[] getResults() throws InterruptedIOException {<a name="line.1231"></a>
+<span class="sourceLineNo">1232</span> waitUntilDone();<a name="line.1232"></a>
+<span class="sourceLineNo">1233</span> return results;<a name="line.1233"></a>
+<span class="sourceLineNo">1234</span> }<a name="line.1234"></a>
+<span class="sourceLineNo">1235</span><a name="line.1235"></a>
+<span class="sourceLineNo">1236</span> /**<a name="line.1236"></a>
+<span class="sourceLineNo">1237</span> * Creates the server error tracker to use inside process.<a name="line.1237"></a>
+<span class="sourceLineNo">1238</span> * Currently, to preserve the main assumption about current retries, and to work well with<a name="line.1238"></a>
+<span class="sourceLineNo">1239</span> * the retry-limit-based calculation, the calculation is local per Process object.<a name="line.1239"></a>
+<span class="sourceLineNo">1240</span> * We may benefit from connection-wide tracking of server errors.<a name="line.1240"></a>
+<span class="sourceLineNo">1241</span> * @return ServerErrorTracker to use, null if there is no ServerErrorTracker on this connection<a name="line.1241"></a>
+<span class="sourceLineNo">1242</span> */<a name="line.1242"></a>
+<span class="sourceLineNo">1243</span> private ConnectionImplementation.ServerErrorTracker createServerErrorTracker() {<a name="line.1243"></a>
+<span class="sourceLineNo">1244</span> return new ConnectionImplementation.ServerErrorTracker(<a name="line.1244"></a>
+<span class="sourceLineNo">1245</span> asyncProcess.serverTrackerTimeout, asyncProcess.numTries);<a name="line.1245"></a>
+<span class="sourceLineNo">1246</span> }<a name="line.1246"></a>
+<span class="sourceLineNo">1247</span><a name="line.1247"></a>
+<span class="sourceLineNo">1248</span> /**<a name="line.1248"></a>
+<span class="sourceLineNo">1249</span> * Create a callable. Isolated to be easily overridden in the tests.<a name="line.1249"></a>
+<span class="sourceLineNo">1250</span> */<a name="line.1250"></a>
+<span class="sourceLineNo">1251</span> private MultiServerCallable createCallable(final ServerName server, TableName tableName,<a name="line.1251"></a>
+<span class="sourceLineNo">1252</span> final MultiAction multi) {<a name="line.1252"></a>
+<span class="sourceLineNo">1253</span> return new MultiServerCallable(asyncProcess.connection, tableName, server,<a name="line.1253"></a>
+<span class="sourceLineNo">1254</span> multi, asyncProcess.rpcFactory.newController(), rpcTimeout, tracker, multi.getPriority());<a name="line.1254"></a>
+<span class="sourceLineNo">1255</span> }<a name="line.1255"></a>
+<span class="sourceLineNo">1256</span><a name="line.1256"></a>
+<span class="sourceLineNo">1257</span> private void updateResult(int index, Object result) {<a name="line.1257"></a>
+<span class="sourceLineNo">1258</span> Object current = results[index];<a name="line.1258"></a>
+<span class="sourceLineNo">1259</span> if (current != null) {<a name="line.1259"></a>
+<span class="sourceLineNo">1260</span> if (LOG.isDebugEnabled()) {<a name="line.1260"></a>
+<span class="sourceLineNo">1261</span> LOG.debug("The result is assigned repeatedly! current:" + current<a name="line.1261"></a>
+<span class="sourceLineNo">1262</span> + ", new:" + result);<a name="line.1262"></a>
+<span class="sourceLineNo">1263</span> }<a name="line.1263"></a>
+<span class="sourceLineNo">1264</span> }<a name="line.1264"></a>
+<span class="sourceLineNo">1265</span> results[index] = result;<a name="line.1265"></a>
+<span class="sourceLineNo">1266</span> }<a name="line.1266"></a>
+<span class="sourceLineNo">1267</span><a name="line.1267"></a>
+<span class="sourceLineNo">1268</span> @VisibleForTesting<a name="line.1268"></a>
+<span class="sourceLineNo">1269</span> long getNumberOfActionsInProgress() {<a name="line.1269"></a>
+<span class="sourceLineNo">1270</span> return actionsInProgress.get();<a name="line.1270"></a>
+<span class="sourceLineNo">1271</span> }<a name="line.1271"></a>
+<span class="sourceLineNo">1272</span>}<a name="line.1272"></a>