http://git-wip-us.apache.org/repos/asf/hbase-site/blob/fa7d6c0c/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/RSRpcServices.html ---------------------------------------------------------------------- diff --git a/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/RSRpcServices.html b/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/RSRpcServices.html index e0b1774..f3d4c5e 100644 --- a/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/RSRpcServices.html +++ b/devapidocs/src-html/org/apache/hadoop/hbase/regionserver/RSRpcServices.html @@ -605,13 +605,13 @@ <span class="sourceLineNo">597</span> * @param row<a name="line.597"></a> <span class="sourceLineNo">598</span> * @param family<a name="line.598"></a> <span class="sourceLineNo">599</span> * @param qualifier<a name="line.599"></a> -<span class="sourceLineNo">600</span> * @param op<a name="line.600"></a> +<span class="sourceLineNo">600</span> * @param compareOp<a name="line.600"></a> <span class="sourceLineNo">601</span> * @param comparator @throws IOException<a name="line.601"></a> <span class="sourceLineNo">602</span> */<a name="line.602"></a> <span class="sourceLineNo">603</span> private boolean checkAndRowMutate(final Region region, final List<ClientProtos.Action> actions,<a name="line.603"></a> -<span class="sourceLineNo">604</span> final CellScanner cellScanner, byte[] row, byte[] family, byte[] qualifier,<a name="line.604"></a> -<span class="sourceLineNo">605</span> CompareOperator op, ByteArrayComparable comparator, RegionActionResult.Builder builder,<a name="line.605"></a> -<span class="sourceLineNo">606</span> ActivePolicyEnforcement spaceQuotaEnforcement) throws IOException {<a name="line.606"></a> +<span class="sourceLineNo">604</span> final CellScanner cellScanner, byte[] row, byte[] family, byte[] qualifier,<a name="line.604"></a> +<span class="sourceLineNo">605</span> CompareOperator op, ByteArrayComparable comparator, RegionActionResult.Builder builder,<a name="line.605"></a> +<span class="sourceLineNo">606</span> ActivePolicyEnforcement spaceQuotaEnforcement) throws IOException {<a name="line.606"></a> <span class="sourceLineNo">607</span> if (!region.getRegionInfo().isMetaTable()) {<a name="line.607"></a> <span class="sourceLineNo">608</span> regionServer.cacheFlusher.reclaimMemStoreMemory();<a name="line.608"></a> <span class="sourceLineNo">609</span> }<a name="line.609"></a> @@ -656,2842 +656,2846 @@ <span class="sourceLineNo">648</span><a name="line.648"></a> <span class="sourceLineNo">649</span> /**<a name="line.649"></a> <span class="sourceLineNo">650</span> * Execute an append mutation.<a name="line.650"></a> -<span class="sourceLineNo">651</span> * @return result to return to client if default operation should be<a name="line.651"></a> -<span class="sourceLineNo">652</span> * bypassed as indicated by RegionObserver, null otherwise<a name="line.652"></a> -<span class="sourceLineNo">653</span> * @throws IOException<a name="line.653"></a> -<span class="sourceLineNo">654</span> */<a name="line.654"></a> -<span class="sourceLineNo">655</span> private Result append(final Region region, final OperationQuota quota,<a name="line.655"></a> -<span class="sourceLineNo">656</span> final MutationProto mutation, final CellScanner cellScanner, long nonceGroup,<a name="line.656"></a> -<span class="sourceLineNo">657</span> ActivePolicyEnforcement spaceQuota)<a name="line.657"></a> -<span class="sourceLineNo">658</span> throws IOException {<a name="line.658"></a> -<span class="sourceLineNo">659</span> long before = EnvironmentEdgeManager.currentTime();<a name="line.659"></a> -<span class="sourceLineNo">660</span> Append append = ProtobufUtil.toAppend(mutation, cellScanner);<a name="line.660"></a> -<span class="sourceLineNo">661</span> checkCellSizeLimit(region, append);<a name="line.661"></a> -<span class="sourceLineNo">662</span> spaceQuota.getPolicyEnforcement(region).check(append);<a name="line.662"></a> -<span class="sourceLineNo">663</span> quota.addMutation(append);<a name="line.663"></a> -<span class="sourceLineNo">664</span> Result r = null;<a name="line.664"></a> -<span class="sourceLineNo">665</span> if (region.getCoprocessorHost() != null) {<a name="line.665"></a> -<span class="sourceLineNo">666</span> r = region.getCoprocessorHost().preAppend(append);<a name="line.666"></a> -<span class="sourceLineNo">667</span> }<a name="line.667"></a> -<span class="sourceLineNo">668</span> if (r == null) {<a name="line.668"></a> -<span class="sourceLineNo">669</span> boolean canProceed = startNonceOperation(mutation, nonceGroup);<a name="line.669"></a> -<span class="sourceLineNo">670</span> boolean success = false;<a name="line.670"></a> -<span class="sourceLineNo">671</span> try {<a name="line.671"></a> -<span class="sourceLineNo">672</span> long nonce = mutation.hasNonce() ? mutation.getNonce() : HConstants.NO_NONCE;<a name="line.672"></a> -<span class="sourceLineNo">673</span> if (canProceed) {<a name="line.673"></a> -<span class="sourceLineNo">674</span> r = region.append(append, nonceGroup, nonce);<a name="line.674"></a> -<span class="sourceLineNo">675</span> } else {<a name="line.675"></a> -<span class="sourceLineNo">676</span> // convert duplicate append to get<a name="line.676"></a> -<span class="sourceLineNo">677</span> List<Cell> results = region.get(ProtobufUtil.toGet(mutation, cellScanner), false,<a name="line.677"></a> -<span class="sourceLineNo">678</span> nonceGroup, nonce);<a name="line.678"></a> -<span class="sourceLineNo">679</span> r = Result.create(results);<a name="line.679"></a> -<span class="sourceLineNo">680</span> }<a name="line.680"></a> -<span class="sourceLineNo">681</span> success = true;<a name="line.681"></a> -<span class="sourceLineNo">682</span> } finally {<a name="line.682"></a> -<span class="sourceLineNo">683</span> if (canProceed) {<a name="line.683"></a> -<span class="sourceLineNo">684</span> endNonceOperation(mutation, nonceGroup, success);<a name="line.684"></a> -<span class="sourceLineNo">685</span> }<a name="line.685"></a> -<span class="sourceLineNo">686</span> }<a name="line.686"></a> -<span class="sourceLineNo">687</span> if (region.getCoprocessorHost() != null) {<a name="line.687"></a> -<span class="sourceLineNo">688</span> r = region.getCoprocessorHost().postAppend(append, r);<a name="line.688"></a> -<span class="sourceLineNo">689</span> }<a name="line.689"></a> -<span class="sourceLineNo">690</span> }<a name="line.690"></a> -<span class="sourceLineNo">691</span> if (regionServer.metricsRegionServer != null) {<a name="line.691"></a> -<span class="sourceLineNo">692</span> regionServer.metricsRegionServer.updateAppend(<a name="line.692"></a> -<span class="sourceLineNo">693</span> EnvironmentEdgeManager.currentTime() - before);<a name="line.693"></a> +<span class="sourceLineNo">651</span> *<a name="line.651"></a> +<span class="sourceLineNo">652</span> * @param region<a name="line.652"></a> +<span class="sourceLineNo">653</span> * @param m<a name="line.653"></a> +<span class="sourceLineNo">654</span> * @param cellScanner<a name="line.654"></a> +<span class="sourceLineNo">655</span> * @return result to return to client if default operation should be<a name="line.655"></a> +<span class="sourceLineNo">656</span> * bypassed as indicated by RegionObserver, null otherwise<a name="line.656"></a> +<span class="sourceLineNo">657</span> * @throws IOException<a name="line.657"></a> +<span class="sourceLineNo">658</span> */<a name="line.658"></a> +<span class="sourceLineNo">659</span> private Result append(final Region region, final OperationQuota quota,<a name="line.659"></a> +<span class="sourceLineNo">660</span> final MutationProto mutation, final CellScanner cellScanner, long nonceGroup,<a name="line.660"></a> +<span class="sourceLineNo">661</span> ActivePolicyEnforcement spaceQuota)<a name="line.661"></a> +<span class="sourceLineNo">662</span> throws IOException {<a name="line.662"></a> +<span class="sourceLineNo">663</span> long before = EnvironmentEdgeManager.currentTime();<a name="line.663"></a> +<span class="sourceLineNo">664</span> Append append = ProtobufUtil.toAppend(mutation, cellScanner);<a name="line.664"></a> +<span class="sourceLineNo">665</span> checkCellSizeLimit(region, append);<a name="line.665"></a> +<span class="sourceLineNo">666</span> spaceQuota.getPolicyEnforcement(region).check(append);<a name="line.666"></a> +<span class="sourceLineNo">667</span> quota.addMutation(append);<a name="line.667"></a> +<span class="sourceLineNo">668</span> Result r = null;<a name="line.668"></a> +<span class="sourceLineNo">669</span> if (region.getCoprocessorHost() != null) {<a name="line.669"></a> +<span class="sourceLineNo">670</span> r = region.getCoprocessorHost().preAppend(append);<a name="line.670"></a> +<span class="sourceLineNo">671</span> }<a name="line.671"></a> +<span class="sourceLineNo">672</span> if (r == null) {<a name="line.672"></a> +<span class="sourceLineNo">673</span> boolean canProceed = startNonceOperation(mutation, nonceGroup);<a name="line.673"></a> +<span class="sourceLineNo">674</span> boolean success = false;<a name="line.674"></a> +<span class="sourceLineNo">675</span> try {<a name="line.675"></a> +<span class="sourceLineNo">676</span> long nonce = mutation.hasNonce() ? mutation.getNonce() : HConstants.NO_NONCE;<a name="line.676"></a> +<span class="sourceLineNo">677</span> if (canProceed) {<a name="line.677"></a> +<span class="sourceLineNo">678</span> r = region.append(append, nonceGroup, nonce);<a name="line.678"></a> +<span class="sourceLineNo">679</span> } else {<a name="line.679"></a> +<span class="sourceLineNo">680</span> // convert duplicate append to get<a name="line.680"></a> +<span class="sourceLineNo">681</span> List<Cell> results = region.get(ProtobufUtil.toGet(mutation, cellScanner), false,<a name="line.681"></a> +<span class="sourceLineNo">682</span> nonceGroup, nonce);<a name="line.682"></a> +<span class="sourceLineNo">683</span> r = Result.create(results);<a name="line.683"></a> +<span class="sourceLineNo">684</span> }<a name="line.684"></a> +<span class="sourceLineNo">685</span> success = true;<a name="line.685"></a> +<span class="sourceLineNo">686</span> } finally {<a name="line.686"></a> +<span class="sourceLineNo">687</span> if (canProceed) {<a name="line.687"></a> +<span class="sourceLineNo">688</span> endNonceOperation(mutation, nonceGroup, success);<a name="line.688"></a> +<span class="sourceLineNo">689</span> }<a name="line.689"></a> +<span class="sourceLineNo">690</span> }<a name="line.690"></a> +<span class="sourceLineNo">691</span> if (region.getCoprocessorHost() != null) {<a name="line.691"></a> +<span class="sourceLineNo">692</span> r = region.getCoprocessorHost().postAppend(append, r);<a name="line.692"></a> +<span class="sourceLineNo">693</span> }<a name="line.693"></a> <span class="sourceLineNo">694</span> }<a name="line.694"></a> -<span class="sourceLineNo">695</span> return r;<a name="line.695"></a> -<span class="sourceLineNo">696</span> }<a name="line.696"></a> -<span class="sourceLineNo">697</span><a name="line.697"></a> -<span class="sourceLineNo">698</span> /**<a name="line.698"></a> -<span class="sourceLineNo">699</span> * Execute an increment mutation.<a name="line.699"></a> -<span class="sourceLineNo">700</span> *<a name="line.700"></a> -<span class="sourceLineNo">701</span> * @param region<a name="line.701"></a> -<span class="sourceLineNo">702</span> * @param mutation<a name="line.702"></a> -<span class="sourceLineNo">703</span> * @return the Result<a name="line.703"></a> -<span class="sourceLineNo">704</span> * @throws IOException<a name="line.704"></a> -<span class="sourceLineNo">705</span> */<a name="line.705"></a> -<span class="sourceLineNo">706</span> private Result increment(final Region region, final OperationQuota quota,<a name="line.706"></a> -<span class="sourceLineNo">707</span> final MutationProto mutation, final CellScanner cells, long nonceGroup,<a name="line.707"></a> -<span class="sourceLineNo">708</span> ActivePolicyEnforcement spaceQuota)<a name="line.708"></a> -<span class="sourceLineNo">709</span> throws IOException {<a name="line.709"></a> -<span class="sourceLineNo">710</span> long before = EnvironmentEdgeManager.currentTime();<a name="line.710"></a> -<span class="sourceLineNo">711</span> Increment increment = ProtobufUtil.toIncrement(mutation, cells);<a name="line.711"></a> -<span class="sourceLineNo">712</span> checkCellSizeLimit(region, increment);<a name="line.712"></a> -<span class="sourceLineNo">713</span> spaceQuota.getPolicyEnforcement(region).check(increment);<a name="line.713"></a> -<span class="sourceLineNo">714</span> quota.addMutation(increment);<a name="line.714"></a> -<span class="sourceLineNo">715</span> Result r = null;<a name="line.715"></a> -<span class="sourceLineNo">716</span> if (region.getCoprocessorHost() != null) {<a name="line.716"></a> -<span class="sourceLineNo">717</span> r = region.getCoprocessorHost().preIncrement(increment);<a name="line.717"></a> -<span class="sourceLineNo">718</span> }<a name="line.718"></a> -<span class="sourceLineNo">719</span> if (r == null) {<a name="line.719"></a> -<span class="sourceLineNo">720</span> boolean canProceed = startNonceOperation(mutation, nonceGroup);<a name="line.720"></a> -<span class="sourceLineNo">721</span> boolean success = false;<a name="line.721"></a> -<span class="sourceLineNo">722</span> try {<a name="line.722"></a> -<span class="sourceLineNo">723</span> long nonce = mutation.hasNonce() ? mutation.getNonce() : HConstants.NO_NONCE;<a name="line.723"></a> -<span class="sourceLineNo">724</span> if (canProceed) {<a name="line.724"></a> -<span class="sourceLineNo">725</span> r = region.increment(increment, nonceGroup, nonce);<a name="line.725"></a> -<span class="sourceLineNo">726</span> } else {<a name="line.726"></a> -<span class="sourceLineNo">727</span> // convert duplicate increment to get<a name="line.727"></a> -<span class="sourceLineNo">728</span> List<Cell> results = region.get(ProtobufUtil.toGet(mutation, cells), false, nonceGroup,<a name="line.728"></a> -<span class="sourceLineNo">729</span> nonce);<a name="line.729"></a> -<span class="sourceLineNo">730</span> r = Result.create(results);<a name="line.730"></a> -<span class="sourceLineNo">731</span> }<a name="line.731"></a> -<span class="sourceLineNo">732</span> success = true;<a name="line.732"></a> -<span class="sourceLineNo">733</span> } finally {<a name="line.733"></a> -<span class="sourceLineNo">734</span> if (canProceed) {<a name="line.734"></a> -<span class="sourceLineNo">735</span> endNonceOperation(mutation, nonceGroup, success);<a name="line.735"></a> -<span class="sourceLineNo">736</span> }<a name="line.736"></a> -<span class="sourceLineNo">737</span> }<a name="line.737"></a> -<span class="sourceLineNo">738</span> if (region.getCoprocessorHost() != null) {<a name="line.738"></a> -<span class="sourceLineNo">739</span> r = region.getCoprocessorHost().postIncrement(increment, r);<a name="line.739"></a> -<span class="sourceLineNo">740</span> }<a name="line.740"></a> -<span class="sourceLineNo">741</span> }<a name="line.741"></a> -<span class="sourceLineNo">742</span> if (regionServer.metricsRegionServer != null) {<a name="line.742"></a> -<span class="sourceLineNo">743</span> regionServer.metricsRegionServer.updateIncrement(<a name="line.743"></a> -<span class="sourceLineNo">744</span> EnvironmentEdgeManager.currentTime() - before);<a name="line.744"></a> +<span class="sourceLineNo">695</span> if (regionServer.metricsRegionServer != null) {<a name="line.695"></a> +<span class="sourceLineNo">696</span> regionServer.metricsRegionServer.updateAppend(<a name="line.696"></a> +<span class="sourceLineNo">697</span> EnvironmentEdgeManager.currentTime() - before);<a name="line.697"></a> +<span class="sourceLineNo">698</span> }<a name="line.698"></a> +<span class="sourceLineNo">699</span> return r;<a name="line.699"></a> +<span class="sourceLineNo">700</span> }<a name="line.700"></a> +<span class="sourceLineNo">701</span><a name="line.701"></a> +<span class="sourceLineNo">702</span> /**<a name="line.702"></a> +<span class="sourceLineNo">703</span> * Execute an increment mutation.<a name="line.703"></a> +<span class="sourceLineNo">704</span> *<a name="line.704"></a> +<span class="sourceLineNo">705</span> * @param region<a name="line.705"></a> +<span class="sourceLineNo">706</span> * @param mutation<a name="line.706"></a> +<span class="sourceLineNo">707</span> * @return the Result<a name="line.707"></a> +<span class="sourceLineNo">708</span> * @throws IOException<a name="line.708"></a> +<span class="sourceLineNo">709</span> */<a name="line.709"></a> +<span class="sourceLineNo">710</span> private Result increment(final Region region, final OperationQuota quota,<a name="line.710"></a> +<span class="sourceLineNo">711</span> final MutationProto mutation, final CellScanner cells, long nonceGroup,<a name="line.711"></a> +<span class="sourceLineNo">712</span> ActivePolicyEnforcement spaceQuota)<a name="line.712"></a> +<span class="sourceLineNo">713</span> throws IOException {<a name="line.713"></a> +<span class="sourceLineNo">714</span> long before = EnvironmentEdgeManager.currentTime();<a name="line.714"></a> +<span class="sourceLineNo">715</span> Increment increment = ProtobufUtil.toIncrement(mutation, cells);<a name="line.715"></a> +<span class="sourceLineNo">716</span> checkCellSizeLimit(region, increment);<a name="line.716"></a> +<span class="sourceLineNo">717</span> spaceQuota.getPolicyEnforcement(region).check(increment);<a name="line.717"></a> +<span class="sourceLineNo">718</span> quota.addMutation(increment);<a name="line.718"></a> +<span class="sourceLineNo">719</span> Result r = null;<a name="line.719"></a> +<span class="sourceLineNo">720</span> if (region.getCoprocessorHost() != null) {<a name="line.720"></a> +<span class="sourceLineNo">721</span> r = region.getCoprocessorHost().preIncrement(increment);<a name="line.721"></a> +<span class="sourceLineNo">722</span> }<a name="line.722"></a> +<span class="sourceLineNo">723</span> if (r == null) {<a name="line.723"></a> +<span class="sourceLineNo">724</span> boolean canProceed = startNonceOperation(mutation, nonceGroup);<a name="line.724"></a> +<span class="sourceLineNo">725</span> boolean success = false;<a name="line.725"></a> +<span class="sourceLineNo">726</span> try {<a name="line.726"></a> +<span class="sourceLineNo">727</span> long nonce = mutation.hasNonce() ? mutation.getNonce() : HConstants.NO_NONCE;<a name="line.727"></a> +<span class="sourceLineNo">728</span> if (canProceed) {<a name="line.728"></a> +<span class="sourceLineNo">729</span> r = region.increment(increment, nonceGroup, nonce);<a name="line.729"></a> +<span class="sourceLineNo">730</span> } else {<a name="line.730"></a> +<span class="sourceLineNo">731</span> // convert duplicate increment to get<a name="line.731"></a> +<span class="sourceLineNo">732</span> List<Cell> results = region.get(ProtobufUtil.toGet(mutation, cells), false, nonceGroup,<a name="line.732"></a> +<span class="sourceLineNo">733</span> nonce);<a name="line.733"></a> +<span class="sourceLineNo">734</span> r = Result.create(results);<a name="line.734"></a> +<span class="sourceLineNo">735</span> }<a name="line.735"></a> +<span class="sourceLineNo">736</span> success = true;<a name="line.736"></a> +<span class="sourceLineNo">737</span> } finally {<a name="line.737"></a> +<span class="sourceLineNo">738</span> if (canProceed) {<a name="line.738"></a> +<span class="sourceLineNo">739</span> endNonceOperation(mutation, nonceGroup, success);<a name="line.739"></a> +<span class="sourceLineNo">740</span> }<a name="line.740"></a> +<span class="sourceLineNo">741</span> }<a name="line.741"></a> +<span class="sourceLineNo">742</span> if (region.getCoprocessorHost() != null) {<a name="line.742"></a> +<span class="sourceLineNo">743</span> r = region.getCoprocessorHost().postIncrement(increment, r);<a name="line.743"></a> +<span class="sourceLineNo">744</span> }<a name="line.744"></a> <span class="sourceLineNo">745</span> }<a name="line.745"></a> -<span class="sourceLineNo">746</span> return r;<a name="line.746"></a> -<span class="sourceLineNo">747</span> }<a name="line.747"></a> -<span class="sourceLineNo">748</span><a name="line.748"></a> -<span class="sourceLineNo">749</span> /**<a name="line.749"></a> -<span class="sourceLineNo">750</span> * Run through the regionMutation <code>rm</code> and per Mutation, do the work, and then when<a name="line.750"></a> -<span class="sourceLineNo">751</span> * done, add an instance of a {@link ResultOrException} that corresponds to each Mutation.<a name="line.751"></a> -<span class="sourceLineNo">752</span> * @param region<a name="line.752"></a> -<span class="sourceLineNo">753</span> * @param actions<a name="line.753"></a> -<span class="sourceLineNo">754</span> * @param cellScanner<a name="line.754"></a> -<span class="sourceLineNo">755</span> * @param builder<a name="line.755"></a> -<span class="sourceLineNo">756</span> * @param cellsToReturn Could be null. May be allocated in this method. This is what this<a name="line.756"></a> -<span class="sourceLineNo">757</span> * method returns as a 'result'.<a name="line.757"></a> -<span class="sourceLineNo">758</span> * @param closeCallBack the callback to be used with multigets<a name="line.758"></a> -<span class="sourceLineNo">759</span> * @param context the current RpcCallContext<a name="line.759"></a> -<span class="sourceLineNo">760</span> * @return Return the <code>cellScanner</code> passed<a name="line.760"></a> -<span class="sourceLineNo">761</span> */<a name="line.761"></a> -<span class="sourceLineNo">762</span> private List<CellScannable> doNonAtomicRegionMutation(final Region region,<a name="line.762"></a> -<span class="sourceLineNo">763</span> final OperationQuota quota, final RegionAction actions, final CellScanner cellScanner,<a name="line.763"></a> -<span class="sourceLineNo">764</span> final RegionActionResult.Builder builder, List<CellScannable> cellsToReturn, long nonceGroup,<a name="line.764"></a> -<span class="sourceLineNo">765</span> final RegionScannersCloseCallBack closeCallBack, RpcCallContext context,<a name="line.765"></a> -<span class="sourceLineNo">766</span> ActivePolicyEnforcement spaceQuotaEnforcement) {<a name="line.766"></a> -<span class="sourceLineNo">767</span> // Gather up CONTIGUOUS Puts and Deletes in this mutations List. Idea is that rather than do<a name="line.767"></a> -<span class="sourceLineNo">768</span> // one at a time, we instead pass them in batch. Be aware that the corresponding<a name="line.768"></a> -<span class="sourceLineNo">769</span> // ResultOrException instance that matches each Put or Delete is then added down in the<a name="line.769"></a> -<span class="sourceLineNo">770</span> // doBatchOp call. We should be staying aligned though the Put and Delete are deferred/batched<a name="line.770"></a> -<span class="sourceLineNo">771</span> List<ClientProtos.Action> mutations = null;<a name="line.771"></a> -<span class="sourceLineNo">772</span> long maxQuotaResultSize = Math.min(maxScannerResultSize, quota.getReadAvailable());<a name="line.772"></a> -<span class="sourceLineNo">773</span> IOException sizeIOE = null;<a name="line.773"></a> -<span class="sourceLineNo">774</span> Object lastBlock = null;<a name="line.774"></a> -<span class="sourceLineNo">775</span> ClientProtos.ResultOrException.Builder resultOrExceptionBuilder = ResultOrException.newBuilder();<a name="line.775"></a> -<span class="sourceLineNo">776</span> boolean hasResultOrException = false;<a name="line.776"></a> -<span class="sourceLineNo">777</span> for (ClientProtos.Action action : actions.getActionList()) {<a name="line.777"></a> -<span class="sourceLineNo">778</span> hasResultOrException = false;<a name="line.778"></a> -<span class="sourceLineNo">779</span> resultOrExceptionBuilder.clear();<a name="line.779"></a> -<span class="sourceLineNo">780</span> try {<a name="line.780"></a> -<span class="sourceLineNo">781</span> Result r = null;<a name="line.781"></a> -<span class="sourceLineNo">782</span><a name="line.782"></a> -<span class="sourceLineNo">783</span> if (context != null<a name="line.783"></a> -<span class="sourceLineNo">784</span> && context.isRetryImmediatelySupported()<a name="line.784"></a> -<span class="sourceLineNo">785</span> && (context.getResponseCellSize() > maxQuotaResultSize<a name="line.785"></a> -<span class="sourceLineNo">786</span> || context.getResponseBlockSize() + context.getResponseExceptionSize()<a name="line.786"></a> -<span class="sourceLineNo">787</span> > maxQuotaResultSize)) {<a name="line.787"></a> -<span class="sourceLineNo">788</span><a name="line.788"></a> -<span class="sourceLineNo">789</span> // We're storing the exception since the exception and reason string won't<a name="line.789"></a> -<span class="sourceLineNo">790</span> // change after the response size limit is reached.<a name="line.790"></a> -<span class="sourceLineNo">791</span> if (sizeIOE == null ) {<a name="line.791"></a> -<span class="sourceLineNo">792</span> // We don't need the stack un-winding do don't throw the exception.<a name="line.792"></a> -<span class="sourceLineNo">793</span> // Throwing will kill the JVM's JIT.<a name="line.793"></a> -<span class="sourceLineNo">794</span> //<a name="line.794"></a> -<span class="sourceLineNo">795</span> // Instead just create the exception and then store it.<a name="line.795"></a> -<span class="sourceLineNo">796</span> sizeIOE = new MultiActionResultTooLarge("Max size exceeded"<a name="line.796"></a> -<span class="sourceLineNo">797</span> + " CellSize: " + context.getResponseCellSize()<a name="line.797"></a> -<span class="sourceLineNo">798</span> + " BlockSize: " + context.getResponseBlockSize());<a name="line.798"></a> -<span class="sourceLineNo">799</span><a name="line.799"></a> -<span class="sourceLineNo">800</span> // Only report the exception once since there's only one request that<a name="line.800"></a> -<span class="sourceLineNo">801</span> // caused the exception. Otherwise this number will dominate the exceptions count.<a name="line.801"></a> -<span class="sourceLineNo">802</span> rpcServer.getMetrics().exception(sizeIOE);<a name="line.802"></a> -<span class="sourceLineNo">803</span> }<a name="line.803"></a> -<span class="sourceLineNo">804</span><a name="line.804"></a> -<span class="sourceLineNo">805</span> // Now that there's an exception is known to be created<a name="line.805"></a> -<span class="sourceLineNo">806</span> // use it for the response.<a name="line.806"></a> -<span class="sourceLineNo">807</span> //<a name="line.807"></a> -<span class="sourceLineNo">808</span> // This will create a copy in the builder.<a name="line.808"></a> -<span class="sourceLineNo">809</span> hasResultOrException = true;<a name="line.809"></a> -<span class="sourceLineNo">810</span> NameBytesPair pair = ResponseConverter.buildException(sizeIOE);<a name="line.810"></a> -<span class="sourceLineNo">811</span> resultOrExceptionBuilder.setException(pair);<a name="line.811"></a> -<span class="sourceLineNo">812</span> context.incrementResponseExceptionSize(pair.getSerializedSize());<a name="line.812"></a> -<span class="sourceLineNo">813</span> resultOrExceptionBuilder.setIndex(action.getIndex());<a name="line.813"></a> -<span class="sourceLineNo">814</span> builder.addResultOrException(resultOrExceptionBuilder.build());<a name="line.814"></a> -<span class="sourceLineNo">815</span> if (cellScanner != null) {<a name="line.815"></a> -<span class="sourceLineNo">816</span> skipCellsForMutation(action, cellScanner);<a name="line.816"></a> -<span class="sourceLineNo">817</span> }<a name="line.817"></a> -<span class="sourceLineNo">818</span> continue;<a name="line.818"></a> -<span class="sourceLineNo">819</span> }<a name="line.819"></a> -<span class="sourceLineNo">820</span> if (action.hasGet()) {<a name="line.820"></a> -<span class="sourceLineNo">821</span> long before = EnvironmentEdgeManager.currentTime();<a name="line.821"></a> -<span class="sourceLineNo">822</span> try {<a name="line.822"></a> -<span class="sourceLineNo">823</span> Get get = ProtobufUtil.toGet(action.getGet());<a name="line.823"></a> -<span class="sourceLineNo">824</span> if (context != null) {<a name="line.824"></a> -<span class="sourceLineNo">825</span> r = get(get, ((HRegion) region), closeCallBack, context);<a name="line.825"></a> -<span class="sourceLineNo">826</span> } else {<a name="line.826"></a> -<span class="sourceLineNo">827</span> r = region.get(get);<a name="line.827"></a> -<span class="sourceLineNo">828</span> }<a name="line.828"></a> -<span class="sourceLineNo">829</span> } finally {<a name="line.829"></a> -<span class="sourceLineNo">830</span> if (regionServer.metricsRegionServer != null) {<a name="line.830"></a> -<span class="sourceLineNo">831</span> regionServer.metricsRegionServer.updateGet(<a name="line.831"></a> -<span class="sourceLineNo">832</span> EnvironmentEdgeManager.currentTime() - before);<a name="line.832"></a> -<span class="sourceLineNo">833</span> }<a name="line.833"></a> -<span class="sourceLineNo">834</span> }<a name="line.834"></a> -<span class="sourceLineNo">835</span> } else if (action.hasServiceCall()) {<a name="line.835"></a> -<span class="sourceLineNo">836</span> hasResultOrException = true;<a name="line.836"></a> -<span class="sourceLineNo">837</span> try {<a name="line.837"></a> -<span class="sourceLineNo">838</span> com.google.protobuf.Message result =<a name="line.838"></a> -<span class="sourceLineNo">839</span> execServiceOnRegion(region, action.getServiceCall());<a name="line.839"></a> -<span class="sourceLineNo">840</span> ClientProtos.CoprocessorServiceResult.Builder serviceResultBuilder =<a name="line.840"></a> -<span class="sourceLineNo">841</span> ClientProtos.CoprocessorServiceResult.newBuilder();<a name="line.841"></a> -<span class="sourceLineNo">842</span> resultOrExceptionBuilder.setServiceResult(<a name="line.842"></a> -<span class="sourceLineNo">843</span> serviceResultBuilder.setValue(<a name="line.843"></a> -<span class="sourceLineNo">844</span> serviceResultBuilder.getValueBuilder()<a name="line.844"></a> -<span class="sourceLineNo">845</span> .setName(result.getClass().getName())<a name="line.845"></a> -<span class="sourceLineNo">846</span> // TODO: Copy!!!<a name="line.846"></a> -<span class="sourceLineNo">847</span> .setValue(UnsafeByteOperations.unsafeWrap(result.toByteArray()))));<a name="line.847"></a> -<span class="sourceLineNo">848</span> } catch (IOException ioe) {<a name="line.848"></a> -<span class="sourceLineNo">849</span> rpcServer.getMetrics().exception(ioe);<a name="line.849"></a> -<span class="sourceLineNo">850</span> NameBytesPair pair = ResponseConverter.buildException(ioe);<a name="line.850"></a> -<span class="sourceLineNo">851</span> resultOrExceptionBuilder.setException(pair);<a name="line.851"></a> -<span class="sourceLineNo">852</span> context.incrementResponseExceptionSize(pair.getSerializedSize());<a name="line.852"></a> -<span class="sourceLineNo">853</span> }<a name="line.853"></a> -<span class="sourceLineNo">854</span> } else if (action.hasMutation()) {<a name="line.854"></a> -<span class="sourceLineNo">855</span> MutationType type = action.getMutation().getMutateType();<a name="line.855"></a> -<span class="sourceLineNo">856</span> if (type != MutationType.PUT && type != MutationType.DELETE && mutations != null &&<a name="line.856"></a> -<span class="sourceLineNo">857</span> !mutations.isEmpty()) {<a name="line.857"></a> -<span class="sourceLineNo">858</span> // Flush out any Puts or Deletes already collected.<a name="line.858"></a> -<span class="sourceLineNo">859</span> doBatchOp(builder, region, quota, mutations, cellScanner, spaceQuotaEnforcement);<a name="line.859"></a> -<span class="sourceLineNo">860</span> mutations.clear();<a name="line.860"></a> -<span class="sourceLineNo">861</span> }<a name="line.861"></a> -<span class="sourceLineNo">862</span> switch (type) {<a name="line.862"></a> -<span class="sourceLineNo">863</span> case APPEND:<a name="line.863"></a> -<span class="sourceLineNo">864</span> r = append(region, quota, action.getMutation(), cellScanner, nonceGroup,<a name="line.864"></a> -<span class="sourceLineNo">865</span> spaceQuotaEnforcement);<a name="line.865"></a> -<span class="sourceLineNo">866</span> break;<a name="line.866"></a> -<span class="sourceLineNo">867</span> case INCREMENT:<a name="line.867"></a> -<span class="sourceLineNo">868</span> r = increment(region, quota, action.getMutation(), cellScanner, nonceGroup,<a name="line.868"></a> +<span class="sourceLineNo">746</span> if (regionServer.metricsRegionServer != null) {<a name="line.746"></a> +<span class="sourceLineNo">747</span> regionServer.metricsRegionServer.updateIncrement(<a name="line.747"></a> +<span class="sourceLineNo">748</span> EnvironmentEdgeManager.currentTime() - before);<a name="line.748"></a> +<span class="sourceLineNo">749</span> }<a name="line.749"></a> +<span class="sourceLineNo">750</span> return r;<a name="line.750"></a> +<span class="sourceLineNo">751</span> }<a name="line.751"></a> +<span class="sourceLineNo">752</span><a name="line.752"></a> +<span class="sourceLineNo">753</span> /**<a name="line.753"></a> +<span class="sourceLineNo">754</span> * Run through the regionMutation <code>rm</code> and per Mutation, do the work, and then when<a name="line.754"></a> +<span class="sourceLineNo">755</span> * done, add an instance of a {@link ResultOrException} that corresponds to each Mutation.<a name="line.755"></a> +<span class="sourceLineNo">756</span> * @param region<a name="line.756"></a> +<span class="sourceLineNo">757</span> * @param actions<a name="line.757"></a> +<span class="sourceLineNo">758</span> * @param cellScanner<a name="line.758"></a> +<span class="sourceLineNo">759</span> * @param builder<a name="line.759"></a> +<span class="sourceLineNo">760</span> * @param cellsToReturn Could be null. May be allocated in this method. This is what this<a name="line.760"></a> +<span class="sourceLineNo">761</span> * method returns as a 'result'.<a name="line.761"></a> +<span class="sourceLineNo">762</span> * @param closeCallBack the callback to be used with multigets<a name="line.762"></a> +<span class="sourceLineNo">763</span> * @param context the current RpcCallContext<a name="line.763"></a> +<span class="sourceLineNo">764</span> * @return Return the <code>cellScanner</code> passed<a name="line.764"></a> +<span class="sourceLineNo">765</span> */<a name="line.765"></a> +<span class="sourceLineNo">766</span> private List<CellScannable> doNonAtomicRegionMutation(final Region region,<a name="line.766"></a> +<span class="sourceLineNo">767</span> final OperationQuota quota, final RegionAction actions, final CellScanner cellScanner,<a name="line.767"></a> +<span class="sourceLineNo">768</span> final RegionActionResult.Builder builder, List<CellScannable> cellsToReturn, long nonceGroup,<a name="line.768"></a> +<span class="sourceLineNo">769</span> final RegionScannersCloseCallBack closeCallBack, RpcCallContext context,<a name="line.769"></a> +<span class="sourceLineNo">770</span> ActivePolicyEnforcement spaceQuotaEnforcement) {<a name="line.770"></a> +<span class="sourceLineNo">771</span> // Gather up CONTIGUOUS Puts and Deletes in this mutations List. Idea is that rather than do<a name="line.771"></a> +<span class="sourceLineNo">772</span> // one at a time, we instead pass them in batch. Be aware that the corresponding<a name="line.772"></a> +<span class="sourceLineNo">773</span> // ResultOrException instance that matches each Put or Delete is then added down in the<a name="line.773"></a> +<span class="sourceLineNo">774</span> // doBatchOp call. We should be staying aligned though the Put and Delete are deferred/batched<a name="line.774"></a> +<span class="sourceLineNo">775</span> List<ClientProtos.Action> mutations = null;<a name="line.775"></a> +<span class="sourceLineNo">776</span> long maxQuotaResultSize = Math.min(maxScannerResultSize, quota.getReadAvailable());<a name="line.776"></a> +<span class="sourceLineNo">777</span> IOException sizeIOE = null;<a name="line.777"></a> +<span class="sourceLineNo">778</span> Object lastBlock = null;<a name="line.778"></a> +<span class="sourceLineNo">779</span> ClientProtos.ResultOrException.Builder resultOrExceptionBuilder = ResultOrException.newBuilder();<a name="line.779"></a> +<span class="sourceLineNo">780</span> boolean hasResultOrException = false;<a name="line.780"></a> +<span class="sourceLineNo">781</span> for (ClientProtos.Action action : actions.getActionList()) {<a name="line.781"></a> +<span class="sourceLineNo">782</span> hasResultOrException = false;<a name="line.782"></a> +<span class="sourceLineNo">783</span> resultOrExceptionBuilder.clear();<a name="line.783"></a> +<span class="sourceLineNo">784</span> try {<a name="line.784"></a> +<span class="sourceLineNo">785</span> Result r = null;<a name="line.785"></a> +<span class="sourceLineNo">786</span><a name="line.786"></a> +<span class="sourceLineNo">787</span> if (context != null<a name="line.787"></a> +<span class="sourceLineNo">788</span> && context.isRetryImmediatelySupported()<a name="line.788"></a> +<span class="sourceLineNo">789</span> && (context.getResponseCellSize() > maxQuotaResultSize<a name="line.789"></a> +<span class="sourceLineNo">790</span> || context.getResponseBlockSize() + context.getResponseExceptionSize()<a name="line.790"></a> +<span class="sourceLineNo">791</span> > maxQuotaResultSize)) {<a name="line.791"></a> +<span class="sourceLineNo">792</span><a name="line.792"></a> +<span class="sourceLineNo">793</span> // We're storing the exception since the exception and reason string won't<a name="line.793"></a> +<span class="sourceLineNo">794</span> // change after the response size limit is reached.<a name="line.794"></a> +<span class="sourceLineNo">795</span> if (sizeIOE == null ) {<a name="line.795"></a> +<span class="sourceLineNo">796</span> // We don't need the stack un-winding do don't throw the exception.<a name="line.796"></a> +<span class="sourceLineNo">797</span> // Throwing will kill the JVM's JIT.<a name="line.797"></a> +<span class="sourceLineNo">798</span> //<a name="line.798"></a> +<span class="sourceLineNo">799</span> // Instead just create the exception and then store it.<a name="line.799"></a> +<span class="sourceLineNo">800</span> sizeIOE = new MultiActionResultTooLarge("Max size exceeded"<a name="line.800"></a> +<span class="sourceLineNo">801</span> + " CellSize: " + context.getResponseCellSize()<a name="line.801"></a> +<span class="sourceLineNo">802</span> + " BlockSize: " + context.getResponseBlockSize());<a name="line.802"></a> +<span class="sourceLineNo">803</span><a name="line.803"></a> +<span class="sourceLineNo">804</span> // Only report the exception once since there's only one request that<a name="line.804"></a> +<span class="sourceLineNo">805</span> // caused the exception. Otherwise this number will dominate the exceptions count.<a name="line.805"></a> +<span class="sourceLineNo">806</span> rpcServer.getMetrics().exception(sizeIOE);<a name="line.806"></a> +<span class="sourceLineNo">807</span> }<a name="line.807"></a> +<span class="sourceLineNo">808</span><a name="line.808"></a> +<span class="sourceLineNo">809</span> // Now that there's an exception is known to be created<a name="line.809"></a> +<span class="sourceLineNo">810</span> // use it for the response.<a name="line.810"></a> +<span class="sourceLineNo">811</span> //<a name="line.811"></a> +<span class="sourceLineNo">812</span> // This will create a copy in the builder.<a name="line.812"></a> +<span class="sourceLineNo">813</span> hasResultOrException = true;<a name="line.813"></a> +<span class="sourceLineNo">814</span> NameBytesPair pair = ResponseConverter.buildException(sizeIOE);<a name="line.814"></a> +<span class="sourceLineNo">815</span> resultOrExceptionBuilder.setException(pair);<a name="line.815"></a> +<span class="sourceLineNo">816</span> context.incrementResponseExceptionSize(pair.getSerializedSize());<a name="line.816"></a> +<span class="sourceLineNo">817</span> resultOrExceptionBuilder.setIndex(action.getIndex());<a name="line.817"></a> +<span class="sourceLineNo">818</span> builder.addResultOrException(resultOrExceptionBuilder.build());<a name="line.818"></a> +<span class="sourceLineNo">819</span> if (cellScanner != null) {<a name="line.819"></a> +<span class="sourceLineNo">820</span> skipCellsForMutation(action, cellScanner);<a name="line.820"></a> +<span class="sourceLineNo">821</span> }<a name="line.821"></a> +<span class="sourceLineNo">822</span> continue;<a name="line.822"></a> +<span class="sourceLineNo">823</span> }<a name="line.823"></a> +<span class="sourceLineNo">824</span> if (action.hasGet()) {<a name="line.824"></a> +<span class="sourceLineNo">825</span> long before = EnvironmentEdgeManager.currentTime();<a name="line.825"></a> +<span class="sourceLineNo">826</span> try {<a name="line.826"></a> +<span class="sourceLineNo">827</span> Get get = ProtobufUtil.toGet(action.getGet());<a name="line.827"></a> +<span class="sourceLineNo">828</span> if (context != null) {<a name="line.828"></a> +<span class="sourceLineNo">829</span> r = get(get, ((HRegion) region), closeCallBack, context);<a name="line.829"></a> +<span class="sourceLineNo">830</span> } else {<a name="line.830"></a> +<span class="sourceLineNo">831</span> r = region.get(get);<a name="line.831"></a> +<span class="sourceLineNo">832</span> }<a name="line.832"></a> +<span class="sourceLineNo">833</span> } finally {<a name="line.833"></a> +<span class="sourceLineNo">834</span> if (regionServer.metricsRegionServer != null) {<a name="line.834"></a> +<span class="sourceLineNo">835</span> regionServer.metricsRegionServer.updateGet(<a name="line.835"></a> +<span class="sourceLineNo">836</span> EnvironmentEdgeManager.currentTime() - before);<a name="line.836"></a> +<span class="sourceLineNo">837</span> }<a name="line.837"></a> +<span class="sourceLineNo">838</span> }<a name="line.838"></a> +<span class="sourceLineNo">839</span> } else if (action.hasServiceCall()) {<a name="line.839"></a> +<span class="sourceLineNo">840</span> hasResultOrException = true;<a name="line.840"></a> +<span class="sourceLineNo">841</span> try {<a name="line.841"></a> +<span class="sourceLineNo">842</span> com.google.protobuf.Message result =<a name="line.842"></a> +<span class="sourceLineNo">843</span> execServiceOnRegion(region, action.getServiceCall());<a name="line.843"></a> +<span class="sourceLineNo">844</span> ClientProtos.CoprocessorServiceResult.Builder serviceResultBuilder =<a name="line.844"></a> +<span class="sourceLineNo">845</span> ClientProtos.CoprocessorServiceResult.newBuilder();<a name="line.845"></a> +<span class="sourceLineNo">846</span> resultOrExceptionBuilder.setServiceResult(<a name="line.846"></a> +<span class="sourceLineNo">847</span> serviceResultBuilder.setValue(<a name="line.847"></a> +<span class="sourceLineNo">848</span> serviceResultBuilder.getValueBuilder()<a name="line.848"></a> +<span class="sourceLineNo">849</span> .setName(result.getClass().getName())<a name="line.849"></a> +<span class="sourceLineNo">850</span> // TODO: Copy!!!<a name="line.850"></a> +<span class="sourceLineNo">851</span> .setValue(UnsafeByteOperations.unsafeWrap(result.toByteArray()))));<a name="line.851"></a> +<span class="sourceLineNo">852</span> } catch (IOException ioe) {<a name="line.852"></a> +<span class="sourceLineNo">853</span> rpcServer.getMetrics().exception(ioe);<a name="line.853"></a> +<span class="sourceLineNo">854</span> NameBytesPair pair = ResponseConverter.buildException(ioe);<a name="line.854"></a> +<span class="sourceLineNo">855</span> resultOrExceptionBuilder.setException(pair);<a name="line.855"></a> +<span class="sourceLineNo">856</span> context.incrementResponseExceptionSize(pair.getSerializedSize());<a name="line.856"></a> +<span class="sourceLineNo">857</span> }<a name="line.857"></a> +<span class="sourceLineNo">858</span> } else if (action.hasMutation()) {<a name="line.858"></a> +<span class="sourceLineNo">859</span> MutationType type = action.getMutation().getMutateType();<a name="line.859"></a> +<span class="sourceLineNo">860</span> if (type != MutationType.PUT && type != MutationType.DELETE && mutations != null &&<a name="line.860"></a> +<span class="sourceLineNo">861</span> !mutations.isEmpty()) {<a name="line.861"></a> +<span class="sourceLineNo">862</span> // Flush out any Puts or Deletes already collected.<a name="line.862"></a> +<span class="sourceLineNo">863</span> doBatchOp(builder, region, quota, mutations, cellScanner, spaceQuotaEnforcement);<a name="line.863"></a> +<span class="sourceLineNo">864</span> mutations.clear();<a name="line.864"></a> +<span class="sourceLineNo">865</span> }<a name="line.865"></a> +<span class="sourceLineNo">866</span> switch (type) {<a name="line.866"></a> +<span class="sourceLineNo">867</span> case APPEND:<a name="line.867"></a> +<span class="sourceLineNo">868</span> r = append(region, quota, action.getMutation(), cellScanner, nonceGroup,<a name="line.868"></a> <span class="sourceLineNo">869</span> spaceQuotaEnforcement);<a name="line.869"></a> <span class="sourceLineNo">870</span> break;<a name="line.870"></a> -<span class="sourceLineNo">871</span> case PUT:<a name="line.871"></a> -<span class="sourceLineNo">872</span> case DELETE:<a name="line.872"></a> -<span class="sourceLineNo">873</span> // Collect the individual mutations and apply in a batch<a name="line.873"></a> -<span class="sourceLineNo">874</span> if (mutations == null) {<a name="line.874"></a> -<span class="sourceLineNo">875</span> mutations = new ArrayList<>(actions.getActionCount());<a name="line.875"></a> -<span class="sourceLineNo">876</span> }<a name="line.876"></a> -<span class="sourceLineNo">877</span> mutations.add(action);<a name="line.877"></a> -<span class="sourceLineNo">878</span> break;<a name="line.878"></a> -<span class="sourceLineNo">879</span> default:<a name="line.879"></a> -<span class="sourceLineNo">880</span> throw new DoNotRetryIOException("Unsupported mutate type: " + type.name());<a name="line.880"></a> -<span class="sourceLineNo">881</span> }<a name="line.881"></a> -<span class="sourceLineNo">882</span> } else {<a name="line.882"></a> -<span class="sourceLineNo">883</span> throw new HBaseIOException("Unexpected Action type");<a name="line.883"></a> -<span class="sourceLineNo">884</span> }<a name="line.884"></a> -<span class="sourceLineNo">885</span> if (r != null) {<a name="line.885"></a> -<span class="sourceLineNo">886</span> ClientProtos.Result pbResult = null;<a name="line.886"></a> -<span class="sourceLineNo">887</span> if (isClientCellBlockSupport(context)) {<a name="line.887"></a> -<span class="sourceLineNo">888</span> pbResult = ProtobufUtil.toResultNoData(r);<a name="line.888"></a> -<span class="sourceLineNo">889</span> // Hard to guess the size here. Just make a rough guess.<a name="line.889"></a> -<span class="sourceLineNo">890</span> if (cellsToReturn == null) {<a name="line.890"></a> -<span class="sourceLineNo">891</span> cellsToReturn = new ArrayList<>();<a name="line.891"></a> -<span class="sourceLineNo">892</span> }<a name="line.892"></a> -<span class="sourceLineNo">893</span> cellsToReturn.add(r);<a name="line.893"></a> -<span class="sourceLineNo">894</span> } else {<a name="line.894"></a> -<span class="sourceLineNo">895</span> pbResult = ProtobufUtil.toResult(r);<a name="line.895"></a> -<span class="sourceLineNo">896</span> }<a name="line.896"></a> -<span class="sourceLineNo">897</span> lastBlock = addSize(context, r, lastBlock);<a name="line.897"></a> -<span class="sourceLineNo">898</span> hasResultOrException = true;<a name="line.898"></a> -<span class="sourceLineNo">899</span> resultOrExceptionBuilder.setResult(pbResult);<a name="line.899"></a> -<span class="sourceLineNo">900</span> }<a name="line.900"></a> -<span class="sourceLineNo">901</span> // Could get to here and there was no result and no exception. Presumes we added<a name="line.901"></a> -<span class="sourceLineNo">902</span> // a Put or Delete to the collecting Mutations List for adding later. In this<a name="line.902"></a> -<span class="sourceLineNo">903</span> // case the corresponding ResultOrException instance for the Put or Delete will be added<a name="line.903"></a> -<span class="sourceLineNo">904</span> // down in the doBatchOp method call rather than up here.<a name="line.904"></a> -<span class="sourceLineNo">905</span> } catch (IOException ie) {<a name="line.905"></a> -<span class="sourceLineNo">906</span> rpcServer.getMetrics().exception(ie);<a name="line.906"></a> -<span class="sourceLineNo">907</span> hasResultOrException = true;<a name="line.907"></a> -<span class="sourceLineNo">908</span> NameBytesPair pair = ResponseConverter.buildException(ie);<a name="line.908"></a> -<span class="sourceLineNo">909</span> resultOrExceptionBuilder.setException(pair);<a name="line.909"></a> -<span class="sourceLineNo">910</span> context.incrementResponseExceptionSize(pair.getSerializedSize());<a name="line.910"></a> -<span class="sourceLineNo">911</span> }<a name="line.911"></a> -<span class="sourceLineNo">912</span> if (hasResultOrException) {<a name="line.912"></a> -<span class="sourceLineNo">913</span> // Propagate index.<a name="line.913"></a> -<span class="sourceLineNo">914</span> resultOrExceptionBuilder.setIndex(action.getIndex());<a name="line.914"></a> -<span class="sourceLineNo">915</span> builder.addResultOrException(resultOrExceptionBuilder.build());<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> // Finish up any outstanding mutations<a name="line.918"></a> -<span class="sourceLineNo">919</span> if (mutations != null && !mutations.isEmpty()) {<a name="line.919"></a> -<span class="sourceLineNo">920</span> doBatchOp(builder, region, quota, mutations, cellScanner, spaceQuotaEnforcement);<a name="line.920"></a> +<span class="sourceLineNo">871</span> case INCREMENT:<a name="line.871"></a> +<span class="sourceLineNo">872</span> r = increment(region, quota, action.getMutation(), cellScanner, nonceGroup,<a name="line.872"></a> +<span class="sourceLineNo">873</span> spaceQuotaEnforcement);<a name="line.873"></a> +<span class="sourceLineNo">874</span> break;<a name="line.874"></a> +<span class="sourceLineNo">875</span> case PUT:<a name="line.875"></a> +<span class="sourceLineNo">876</span> case DELETE:<a name="line.876"></a> +<span class="sourceLineNo">877</span> // Collect the individual mutations and apply in a batch<a name="line.877"></a> +<span class="sourceLineNo">878</span> if (mutations == null) {<a name="line.878"></a> +<span class="sourceLineNo">879</span> mutations = new ArrayList<>(actions.getActionCount());<a name="line.879"></a> +<span class="sourceLineNo">880</span> }<a name="line.880"></a> +<span class="sourceLineNo">881</span> mutations.add(action);<a name="line.881"></a> +<span class="sourceLineNo">882</span> break;<a name="line.882"></a> +<span class="sourceLineNo">883</span> default:<a name="line.883"></a> +<span class="sourceLineNo">884</span> throw new DoNotRetryIOException("Unsupported mutate type: " + type.name());<a name="line.884"></a> +<span class="sourceLineNo">885</span> }<a name="line.885"></a> +<span class="sourceLineNo">886</span> } else {<a name="line.886"></a> +<span class="sourceLineNo">887</span> throw new HBaseIOException("Unexpected Action type");<a name="line.887"></a> +<span class="sourceLineNo">888</span> }<a name="line.888"></a> +<span class="sourceLineNo">889</span> if (r != null) {<a name="line.889"></a> +<span class="sourceLineNo">890</span> ClientProtos.Result pbResult = null;<a name="line.890"></a> +<span class="sourceLineNo">891</span> if (isClientCellBlockSupport(context)) {<a name="line.891"></a> +<span class="sourceLineNo">892</span> pbResult = ProtobufUtil.toResultNoData(r);<a name="line.892"></a> +<span class="sourceLineNo">893</span> // Hard to guess the size here. Just make a rough guess.<a name="line.893"></a> +<span class="sourceLineNo">894</span> if (cellsToReturn == null) {<a name="line.894"></a> +<span class="sourceLineNo">895</span> cellsToReturn = new ArrayList<>();<a name="line.895"></a> +<span class="sourceLineNo">896</span> }<a name="line.896"></a> +<span class="sourceLineNo">897</span> cellsToReturn.add(r);<a name="line.897"></a> +<span class="sourceLineNo">898</span> } else {<a name="line.898"></a> +<span class="sourceLineNo">899</span> pbResult = ProtobufUtil.toResult(r);<a name="line.899"></a> +<span class="sourceLineNo">900</span> }<a name="line.900"></a> +<span class="sourceLineNo">901</span> lastBlock = addSize(context, r, lastBlock);<a name="line.901"></a> +<span class="sourceLineNo">902</span> hasResultOrException = true;<a name="line.902"></a> +<span class="sourceLineNo">903</span> resultOrExceptionBuilder.setResult(pbResult);<a name="line.903"></a> +<span class="sourceLineNo">904</span> }<a name="line.904"></a> +<span class="sourceLineNo">905</span> // Could get to here and there was no result and no exception. Presumes we added<a name="line.905"></a> +<span class="sourceLineNo">906</span> // a Put or Delete to the collecting Mutations List for adding later. In this<a name="line.906"></a> +<span class="sourceLineNo">907</span> // case the corresponding ResultOrException instance for the Put or Delete will be added<a name="line.907"></a> +<span class="sourceLineNo">908</span> // down in the doBatchOp method call rather than up here.<a name="line.908"></a> +<span class="sourceLineNo">909</span> } catch (IOException ie) {<a name="line.909"></a> +<span class="sourceLineNo">910</span> rpcServer.getMetrics().exception(ie);<a name="line.910"></a> +<span class="sourceLineNo">911</span> hasResultOrException = true;<a name="line.911"></a> +<span class="sourceLineNo">912</span> NameBytesPair pair = ResponseConverter.buildException(ie);<a name="line.912"></a> +<span class="sourceLineNo">913</span> resultOrExceptionBuilder.setException(pair);<a name="line.913"></a> +<span class="sourceLineNo">914</span> context.incrementResponseExceptionSize(pair.getSerializedSize());<a name="line.914"></a> +<span class="sourceLineNo">915</span> }<a name="line.915"></a> +<span class="sourceLineNo">916</span> if (hasResultOrException) {<a name="line.916"></a> +<span class="sourceLineNo">917</span> // Propagate index.<a name="line.917"></a> +<span class="sourceLineNo">918</span> resultOrExceptionBuilder.setIndex(action.getIndex());<a name="line.918"></a> +<span class="sourceLineNo">919</span> builder.addResultOrException(resultOrExceptionBuilder.build());<a name="line.919"></a> +<span class="sourceLineNo">920</span> }<a name="line.920"></a> <span class="sourceLineNo">921</span> }<a name="line.921"></a> -<span class="sourceLineNo">922</span> return cellsToReturn;<a name="line.922"></a> -<span class="sourceLineNo">923</span> }<a name="line.923"></a> -<span class="sourceLineNo">924</span><a name="line.924"></a> -<span class="sourceLineNo">925</span> private void checkCellSizeLimit(final Region region, final Mutation m) throws IOException {<a name="line.925"></a> -<span class="sourceLineNo">926</span> if (!(region instanceof HRegion)) {<a name="line.926"></a> -<span class="sourceLineNo">927</span> return;<a name="line.927"></a> -<span class="sourceLineNo">928</span> }<a name="line.928"></a> -<span class="sourceLineNo">929</span> HRegion r = (HRegion)region;<a name="line.929"></a> -<span class="sourceLineNo">930</span> if (r.maxCellSize > 0) {<a name="line.930"></a> -<span class="sourceLineNo">931</span> CellScanner cells = m.cellScanner();<a name="line.931"></a> -<span class="sourceLineNo">932</span> while (cells.advance()) {<a name="line.932"></a> -<span class="sourceLineNo">933</span> int size = CellUtil.estimatedSerializedSizeOf(cells.current());<a name="line.933"></a> -<span class="sourceLineNo">934</span> if (size > r.maxCellSize) {<a name="line.934"></a> -<span class="sourceLineNo">935</span> String msg = "Cell with size " + size + " exceeds limit of " + r.maxCellSize + " bytes";<a name="line.935"></a> -<span class="sourceLineNo">936</span> if (LOG.isDebugEnabled()) {<a name="line.936"></a> -<span class="sourceLineNo">937</span> LOG.debug(msg);<a name="line.937"></a> -<span class="sourceLineNo">938</span> }<a name="line.938"></a> -<span class="sourceLineNo">939</span> throw new DoNotRetryIOException(msg);<a name="line.939"></a> -<span class="sourceLineNo">940</span> }<a name="line.940"></a> -<span class="sourceLineNo">941</span> }<a name="line.941"></a> -<span class="sourceLineNo">942</span> }<a name="line.942"></a> -<span class="sourceLineNo">943</span> }<a name="line.943"></a> -<span class="sourceLineNo">944</span><a name="line.944"></a> -<span class="sourceLineNo">945</span> /**<a name="line.945"></a> -<span class="sourceLineNo">946</span> * Execute a list of Put/Delete mutations.<a name="line.946"></a> -<span class="sourceLineNo">947</span> *<a name="line.947"></a> -<span class="sourceLineNo">948</span> * @param builder<a name="line.948"></a> -<span class="sourceLineNo">949</span> * @param region<a name="line.949"></a> -<span class="sourceLineNo">950</span> * @param mutations<a name="line.950"></a> -<span class="sourceLineNo">951</span> */<a name="line.951"></a> -<span class="sourceLineNo">952</span> private void doBatchOp(final RegionActionResult.Builder builder, final Region region,<a name="line.952"></a> -<span class="sourceLineNo">953</span> final OperationQuota quota, final List<ClientProtos.Action> mutations,<a name="line.953"></a> -<span class="sourceLineNo">954</span> final CellScanner cells, ActivePolicyEnforcement spaceQuotaEnforcement) {<a name="line.954"></a> -<span class="sourceLineNo">955</span> Mutation[] mArray = new Mutation[mutations.size()];<a name="line.955"></a> -<span class="sourceLineNo">956</span> long before = EnvironmentEdgeManager.currentTime();<a name="line.956"></a> -<span class="sourceLineNo">957</span> boolean batchContainsPuts = false, batchContainsDelete = false;<a name="line.957"></a> -<span class="sourceLineNo">958</span> try {<a name="line.958"></a> -<span class="sourceLineNo">959</span> /** HBASE-17924<a name="line.959"></a> -<span class="sourceLineNo">960</span> * mutationActionMap is a map to map the relation between mutations and actions<a name="line.960"></a> -<span class="sourceLineNo">961</span> * since mutation array may have been reoredered.In order to return the right<a name="line.961"></a> -<span class="sourceLineNo">962</span> * result or exception to the corresponding actions, We need to know which action<a name="line.962"></a> -<span class="sourceLineNo">963</span> * is the mutation belong to. We can't sort ClientProtos.Action array, since they<a name="line.963"></a> -<span class="sourceLineNo">964</span> * are bonded to cellscanners.<a name="line.964"></a> -<span class="sourceLineNo">965</span> */<a name="line.965"></a> -<span class="sourceLineNo">966</span> Map<Mutation, ClientProtos.Action> mutationActionMap = new HashMap<Mutation, ClientProtos.Action>();<a name="line.966"></a> -<span class="sourceLineNo">967</span> int i = 0;<a name="line.967"></a> -<span class="sourceLineNo">968</span> for (ClientProtos.Action action: mutations) {<a name="line.968"></a> -<span class="sourceLineNo">969</span> MutationProto m = action.getMutation();<a name="line.969"></a> -<span class="sourceLineNo">970</span> Mutation mutation;<a name="line.970"></a> -<span class="sourceLineNo">971</span> if (m.getMutateType() == MutationType.PUT) {<a name="line.971"></a> -<span class="sourceLineNo">972</span> mutation = ProtobufUtil.toPut(m, cells);<a name="line.972"></a> -<span class="sourceLineNo">973</span> batchContainsPuts = true;<a name="line.973"></a> -<span class="sourceLineNo">974</span> } else {<a name="line.974"></a> -<span class="sourceLineNo">975</span> mutation = ProtobufUtil.toDelete(m, cells);<a name="line.975"></a> -<span class="sourceLineNo">976</span> batchContainsDelete = true;<a name="line.976"></a> -<span class="sourceLineNo">977</span> }<a name="line.977"></a> -<span class="sourceLineNo">978</span> mutationActionMap.put(mutation, action);<a name="line.978"></a> -<span class="sourceLineNo">979</span> mArray[i++] = mutation;<a name="line.979"></a> -<span class="sourceLineNo">980</span> checkCellSizeLimit(region, mutation);<a name="line.980"></a> -<span class="sourceLineNo">981</span> // Check if a space quota disallows this mutation<a name="line.981"></a> -<span class="sourceLineNo">982</span> spaceQuotaEnforcement.getPolicyEnforcement(region).check(mutation);<a name="line.982"></a> -<span class="sourceLineNo">983</span> quota.addMutation(mutation);<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> if (!region.getRegionInfo().isMetaTable()) {<a name="line.986"></a> -<span class="sourceLineNo">987</span> regionServer.cacheFlusher.reclaimMemStoreMemory();<a name="line.987"></a> +<span class="sourceLineNo">922</span> // Finish up any outstanding mutations<a name="line.922"></a> +<span class="sourceLineNo">923</span> if (mutations != null && !mutations.isEmpty()) {<a name="line.923"></a> +<span class="sourceLineNo">924</span> doBatchOp(builder, region, quota, mutations, cellScanner, spaceQuotaEnforcement);<a name="line.924"></a> +<span class="sourceLineNo">925</span> }<a name="line.925"></a> +<span class="sourceLineNo">926</span> return cellsToReturn;<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> private void checkCellSizeLimit(final Region region, final Mutation m) throws IOException {<a name="line.929"></a> +<span class="sourceLineNo">930</span> if (!(region instanceof HRegion)) {<a name="line.930"></a> +<span class="sourceLineNo">931</span> return;<a name="line.931"></a> +<span class="sourceLineNo">932</span> }<a name="line.932"></a> +<span class="sourceLineNo">933</span> HRegion r = (HRegion)region;<a name="line.933"></a> +<span class="sourceLineNo">934</span> if (r.maxCellSize > 0) {<a name="line.934"></a> +<span class="sourceLineNo">935</span> CellScanner cells = m.cellScanner();<a name="line.935"></a> +<span class="sourceLineNo">936</span> while (cells.advance()) {<a name="line.936"></a> +<span class="sourceLineNo">937</span> int size = CellUtil.estimatedSerializedSizeOf(cells.current());<a name="line.937"></a> +<span class="sourceLineNo">938</span> if (size > r.maxCellSize) {<a name="line.938"></a> +<span class="sourceLineNo">939</span> String msg = "Cell with size " + size + " exceeds limit of " + r.maxCellSize + " bytes";<a name="line.939"></a> +<span class="sourceLineNo">940</span> if (LOG.isDebugEnabled()) {<a name="line.940"></a> +<span class="sourceLineNo">941</span> LOG.debug(msg);<a name="line.941"></a> +<span class="sourceLineNo">942</span> }<a name="line.942"></a> +<span class="sourceLineNo">943</span> throw new DoNotRetryIOException(msg);<a name="line.943"></a> +<span class="sourceLineNo">944</span> }<a name="line.944"></a> +<span class="sourceLineNo">945</span> }<a name="line.945"></a> +<span class="sourceLineNo">946</span> }<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> * Execute a list of Put/Delete mutations.<a name="line.950"></a> +<span class="sourceLineNo">951</span> *<a name="line.951"></a> +<span class="sourceLineNo">952</span> * @param builder<a name="line.952"></a> +<span class="sourceLineNo">953</span> * @param region<a name="line.953"></a> +<span class="sourceLineNo">954</span> * @param mutations<a name="line.954"></a> +<span class="sourceLineNo">955</span> */<a name="line.955"></a> +<span class="sourceLineNo">956</span> private void doBatchOp(final RegionActionResult.Builder builder, final Region region,<a name="line.956"></a> +<span class="sourceLineNo">957</span> final OperationQuota quota, final List<ClientProtos.Action> mutations,<a name="line.957"></a> +<span class="sourceLineNo">958</span> final CellScanner cells, ActivePolicyEnforcement spaceQuotaEnforcement) {<a name="line.958"></a> +<span class="sourceLineNo">959</span> Mutation[] mArray = new Mutation[mutations.size()];<a name="line.959"></a> +<span class="sourceLineNo">960</span> long before = EnvironmentEdgeManager.currentTime();<a name="line.960"></a> +<span class="sourceLineNo">961</span> boolean batchContainsPuts = false, batchContainsDelete = false;<a name="line.961"></a> +<span class="sourceLineNo">962</span> try {<a name="line.962"></a> +<span class="sourceLineNo">963</span> /** HBASE-17924<a name="line.963"></a> +<span class="sourceLineNo">964</span> * mutationActionMap is a map to map the relation between mutations and actions<a name="line.964"></a> +<span class="sourceLineNo">965</span> * since mutation array may have been reoredered.In order to return the right<a name="line.965"></a> +<span class="sourceLineNo">966</span> * result or exception to the corresponding actions, We need to know which action<a name="line.966"></a> +<span class="sourceLineNo">967</span> * is the mutation belong to. We can't sort ClientProtos.Action array, since they<a name="line.967"></a> +<span class="sourceLineNo">968</span> * are bonded to cellscanners.<a name="line.968"></a> +<span class="sourceLineNo">969</span> */<a name="line.969"></a> +<span class="sourceLineNo">970</span> Map<Mutation, ClientProtos.Action> mutationActionMap = new HashMap<Mutation, ClientProtos.Action>();<a name="line.970"></a> +<span class="sourceLineNo">971</span> int i = 0;<a name="line.971"></a> +<span class="sourceLineNo">972</span> for (ClientProtos.Action action: mutations) {<a name="line.972"></a> +<span class="sourceLineNo">973</span> MutationProto m = action.getMutation();<a name="line.973"></a> +<span class="sourceLineNo">974</span> Mutation mutation;<a name="line.974"></a> +<span class="sourceLineNo">975</span> if (m.getMutateType() == MutationType.PUT) {<a name="line.975"></a> +<span class="sourceLineNo">976</span> mutation = ProtobufUtil.toPut(m, cells);<a name="line.976"></a> +<span class="sourceLineNo">977</span> batchContainsPuts = true;<a name="line.977"></a> +<span class="sourceLineNo">978</span> } else {<a name="line.978"></a> +<span class="sourceLineNo">979</span> mutation = ProtobufUtil.toDelete(m, cells);<a name="line.979"></a> +<span class="sourceLineNo">980</span> batchContainsDelete = true;<a name="line.980"></a> +<span class="sourceLineNo">981</span> }<a name="line.981"></a> +<span class="sourceLineNo">982</span> mutationActionMap.put(mutation, action);<a name="line.982"></a> +<span class="sourceLineNo">983</span> mArray[i++] = mutation;<a name="line.983"></a> +<span class="sourceLineNo">984</span> checkCellSizeLimit(region, mutation);<a name="line.984"></a> +<span class="sourceLineNo">985</span> // Check if a space quota disallows this mutation<a name="line.985"></a> +<span class="sourceLineNo">986</span> spaceQuotaEnforcement.getPolicyEnforcement(region).check(mutation);<a name="line.986"></a> +<span class="sourceLineNo">987</span> quota.addMutation(mutation);<a name="line.987"></a> <span class="sourceLineNo">988</span> }<a name="line.988"></a> <span class="sourceLineNo">989</span><a name="line.989"></a> -<span class="sourceLineNo">990</span> // HBASE-17924<a name="line.990"></a> -<span class="sourceLineNo">991</span> // sort to improve lock efficiency<a name="line.991"></a> -<span class="sourceLineNo">992</span> Arrays.sort(mArray);<a name="line.992"></a> +<span class="sourceLineNo">990</span> if (!region.getRegionInfo().isMetaTable()) {<a name="line.990"></a> +<span class="sourceLineNo">991</span> regionServer.cacheFlusher.reclaimMemStoreMemory();<a name="line.991"></a> +<span class="sourceLineNo">992</span> }<a name="line.992"></a> <span class="sourceLineNo">993</span><a name="line.993"></a> -<span class="sourceLineNo">994</span> OperationStatus[] codes = region.batchMutate(mArray, HConstants.NO_NONCE,<a name="line.994"></a> -<span class="sourceLineNo">995</span> HConstants.NO_NONCE);<a name="line.995"></a> -<span class="sourceLineNo">996</span> for (i = 0; i < codes.length; i++) {<a name="line.996"></a> -<span class="sourceLineNo">997</span> Mutation currentMutation = mArray[i];<a name="line.997"></a> -<span class="sourceLineNo">998</span> ClientProtos.Action currentAction = mutationActionMap.get(currentMutation);<a name="line.998"></a> -<span class="sourceLineNo">999</span> int index = currentAction.getIndex();<a name="line.999"></a> -<span class="sourceLineNo">1000</span> Exception e = null;<a name="line.1000"></a> -<span class="sourceLineNo">1001</span> switch (codes[i].getOperationStatusCode()) {<a name="line.1001"></a> -<span class="sourceLineNo">1002</span> case BAD_FAMILY:<a name="line.1002"></a> -<span class="sourceLineNo">1003</span> e = new NoSuchColumnFamilyException(codes[i].getExceptionMsg());<a name="line.1003"></a> -<span class="sourceLineNo">1004</span> builder.addResultOrException(getResultOrException(e, index));<a name="line.1004"></a> -<span class="sourceLineNo">1005</span> break;<a name="line.1005"></a> -<span class="sourceLineNo">1006</span><a name="line.1006"></a> -<span class="sourceLineNo">1007</span> case SANITY_CHECK_FAILURE:<a name="line.1007"></a> -<span class="sourceLineNo">1008</span> e = new FailedSanityCheckException(codes[i].getExceptionMsg());<a name="line.1008"></a> -<span class="sourceLineNo">1009</span> builder.addResultOrException(getResultOrException(e, index));<a name="line.1009"></a> -<span class="sourceLineNo">1010</span> break;<a name="line.1010"></a> -<span class="sourceLineNo">1011</span><a name="line.1011"></a> -<span class="sourceLineNo">1012</span> default:<a name="line.1012"></a> -<span class="sourceLineNo">1013</span> e = new DoNotRetryIOException(codes[i].getExceptionMsg());<a name="line.1013"></a> -<span class="sourceLineNo">1014</span> builder.addResultOrException(getResultOrException(e, index));<a name="line.1014"></a> -<span class="sourceLineNo">1015</span> break;<a name="line.1015"></a> -<span class="sourceLineNo">1016</span><a name="line.1016"></a> -<span class="sourceLineNo">1017</span> case SUCCESS:<a name="line.1017"></a> -<span class="sourceLineNo">1018</span> builder.addResultOrException(getResultOrException(<a name="line.1018"></a> -<span class="sourceLineNo">1019</span> ClientProtos.Result.getDefaultInstance(), index));<a name="line.1019"></a> -<span class="sourceLineNo">1020</span> break;<a name="line.1020"></a> -<span class="sourceLineNo">1021</span> }<a name="line.1021"></a> -<span class="sourceLineNo">1022</span> }<a name="line.1022"></a> -<span class="sourceLineNo">1023</span> } catch (IOException ie) {<a name="line.1023"></a> -<span class="sourceLineNo">1024</span> for (int i = 0; i < mutations.size(); i++) {<a name="line.1024"></a> -<span class="sourceLineNo">1025</span> builder.addResultOrException(getResultOrException(ie, mutations.get(i).getIndex()));<a name="line.1025"></a> +<span class="sourceLineNo">994</span> // HBASE-17924<a name="line.994"></a> +<span class="sourceLineNo">995</span> // sort to improve lock efficiency<a name="line.995"></a> +<span class="sourceLineNo">996</span> Arrays.sort(mArray);<a name="line.996"></a> +<span class="sourceLineNo">997</span><a name="line.997"></a> +<span class="sourceLineNo">998</span> OperationStatus[] codes = region.batchMutate(mArray, HConstants.NO_NONCE,<a name="line.998"></a> +<span class="sourceLineNo">999</span> HConstants.NO_NONCE);<a name="line.999"></a> +<span class="sourceLineNo">1000</span> for (i = 0; i < codes.length; i++) {<a name="line.1000"></a> +<span class="sourceLineNo">1001</span> Mutation currentMutation = mArray[i];<a name="line.1001"></a> +<span class="sourceLineNo">1002</span> ClientProtos.Action currentAction = mutationActionMap.get(currentMutation);<a name="line.1002"></a> +<span class="sourceLineNo">1003</span> int index = currentAction.getIndex();<a name="line.1003"></a> +<span class="sourceLineNo">1004</span> Exception e = null;<a name="line.1004"></a> +<span class="sourceLineNo">1005</span> switch (codes[i].getOperationStatusCode()) {<a name="line.1005"></a> +<span class="sourceLineNo">1006</span> case BAD_FAMILY:<a name="line.1006"></a> +<span class="sourceLineNo">1007</span> e = new NoSuchColumnFamilyException(codes[i].getExceptionMsg());<a name="line.1007"></a> +<span class="sourceLineNo">1008</span> builder.addResultOrException(getResultOrException(e, index));<a name="line.1008"></a> +<span class="sourceLineNo">1009</span> break;<a name="line.1009"></a> +<span class="sourceLineNo">1010</span><a name="line.1010"></a> +<span class="sourceLineNo">1011</span> case SANITY_CHECK_FAILURE:<a name="line.1011"></a> +<span class="sourceLineNo">1012</span> e = new FailedSanityCheckException(codes[i].getExceptionMsg());<a name="line.1012"></a> +<span class="sourceLineNo">1013</span> builder.addResultOrException(getResultOrException(e, index));<a name="line.1013"></a> +<span class="sourceLineNo">1014</span> break;<a name="line.1014"></a> +<span class="sourceLineNo">1015</span><a name="line.1015"></a> +<span class="sourceLineNo">1016</span> default:<a name="line.1016"></a> +<span class="sourceLineNo">1017</span> e = new DoNotRetryIOException(codes[i].getExceptionMsg());<a name="line.1017"></a> +<span class="sourceLineNo">1018</span> builder.addResultOrException(getResultOrException(e, index));<a name="line.1018"></a> +<span class="sourceLineNo">1019</span> break;<a name="line.1019"></a> +<span class="sourceLineNo">1020</span><a name="line.1020"></a> +<span class="sourceLineNo">1021</span> case SUCCESS:<a name="line.1021"></a> +<span class="sourceLineNo">1022</span> builder.addResultOrException(getResultOrException(<a name="line.1022"></a> +<span class="sourceLineNo">1023</span> ClientProtos.Result.getDefaultInstance(), index));<a name="line.1023"></a> +<span class="sourceLineNo">1024</span> break;<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> if (regionServer.metricsRegionServer != null) {<a name="line.1028"></a> -<span class="sourceLineNo">1029</span> long after = EnvironmentEdgeManager.currentTime();<a name="line.1029"></a> -<span class="sourceLineNo">1030</span> if (batchContainsPuts) {<a name="line.1030"></a> -<span class="sourceLineNo">1031</span> regionServer.metricsRegionServer.updatePutBatch(after - before);<a name="line.1031"></a> -<span class="sourceLineNo">1032</span> }<a name="line.1032"></a> -<span class="sourceLineNo">1033</span> if (batchContainsDelete) {<a name="line.1033"></a> -<span class="sourceLineNo">1034</span> regionServer.metricsRegionServer.updateDeleteBatch(after - before);<a name="line.1034"></a> -<span class="sourceLineNo">1035</span> }<a name="line.1035"></a> -<span class="sourceLineNo">1036</span> }<a name="line.1036"></a> -<span class="sourceLineNo">1037</span> }<a name="line.1037"></a> -<span class="sourceLineNo">1038</span><a name="line.1038"></a> -<span class="sourceLineNo">1039</span> /**<a name="line.1039"></a> -<span class="sourceLineNo">1040</span> * Execute a list of Put/Delete mutations. The function returns OperationStatus instead of<a name="line.1040"></a> -<span class="sourceLineNo">1041</span> * constructing MultiResponse to save a possible loop if caller doesn't need MultiResponse.<a name="line.1041"></a> -<span class="sourceLineNo">1042</span> * @param region<a name="line.1042"></a> -<span class="sourceLineNo">1043</span> * @param mutations<a name="line.1043"></a> -<span class="sourceLineNo">1044</span> * @param replaySeqId<a name="line.1044"></a> -<span class="sourceLineNo">1045</span> * @return an array of OperationStatus which internally contains the OperationStatusCode and the<a name="line.1045"></a> -<span class="sourceLineNo">1046</span> * exceptionMessage if any<a name="line.1046"></a> -<span class="sourceLineNo">1047</span> * @throws IOException<a name="line.1047"></a> -<span class="sourceLineNo">1048</span> */<a name="line.1048"></a> -<span class="sourceLineNo">1049</span> private OperationStatus [] doReplayBatchOp(final Region region,<a name="line.1049"></a> -<span class="sourceLineNo">1050</span> final List<WALSplitter.MutationReplay> mutations, long replaySeqId) throws IOException {<a name="line.1050"></a> -<span class="sourceLineNo">1051</span> long before = EnvironmentEdgeManager.currentTime();<a name="line.1051"></a> -<span class="sourceLineNo">1052</span> boolean batchContainsPuts = false, batchContainsDelete = false;<a name="line.1052"></a> -<span class="sourceLineNo">1053</span> try {<a name="line.1053"></a> -<span class="sourceLineNo">1054</span> for (Iterator<WALSplitter.MutationReplay> it = mutations.iterator(); it.hasNext();) {<a name="line.1054"></a> -<span class="sourceLineNo">1055</span> WALSplitter.MutationReplay m = it.next();<a name="line.1055"></a> -<span class="sourceLineNo">1056</span><a name="line.1056"></a> -<span class="sourceLineNo">1057</span> if (m.type == MutationType.PUT) {<a name="line.1057"></a> -<span class="sourceLineNo">1058</span> batchContainsPuts = true;<a name="line.1058"></a> -<span class="sourceLineNo">1059</span> } else {<a name="line.1059"></a> -<span class="sourceLineNo">1060</span> batchContainsDelete = true;<a name="line.1060"></a> -<span class="sourceLineNo">1061</span> }<a name="line.1061"></a> -<span class="sourceLineNo">1062</span><a name="line.1062"></a> -<span class="sourceLineNo">1063</span> NavigableMap<byte[], List<Cell>> map = m.mutation.getFamilyCellMap();<a name="line.1063"></a> -<span class="sourceLineNo">1064</span> List<Cell> metaCells = map.get(WALEdit.METAFAMILY);<a name="line.1064"></a> -<span class="sourceLineNo">1065</span> if (metaCells != null && !metaCells.isEmpty()) {<a name="line.1065"></a> -<span class="sourceLineNo">1066</span> for (Cell metaCell : metaCells) {<a name="line.1066"></a> -<span class="sourceLineNo">1067</span> CompactionDescriptor compactionDesc = WALEdit.getCompaction(metaCell);<a name="line.1067"></a> -<span class="sourceLineNo">1068</span> boolean isDefaultReplica = RegionReplicaUtil.isDefaultReplica(region.getRegionInfo());<a name="line.1068"></a> -<span class="sourceLineNo">1069</span> HRegion hRegion = (HRegion)region;<a name="line.1069"></a> -<span class="sourceLineNo">1070</span> if (compactionDesc != null) {<a name="line.1070"></a> -<span class="sourceLineNo">1071</span> // replay the compaction. Remove the files from stores only if we are the primary<a name="line.1071"></a> -<span class="sourceLineNo">1072</span> // region replica (thus own the files)<a name="line.1072"></a> -<span class="sourceLineNo">1073</span> hRegion.replayWALCompactionMarker(compactionDesc, !isDefaultReplica, isDefaultReplica,<a name="line.1073"></a> -<span class="sourceLineNo">1074</span> replaySeqId);<a name="line.1074"></a> -<span class="sourceLineNo">1075</span> continue;<a name="line.1075"></a> -<span class="sourceLineNo">1076</span> }<a name="line.1076"></a> -<span class="sourceLineNo">1077</span> FlushDescriptor flushDesc = WALEdit.getFlushDescriptor(metaCell);<a name="line.1077"></a> -<span class="sourceLineNo">1078</span> if (flushDesc != null && !isDefaultReplica) {<a name="line.1078"></a> -<span class="sourceLineNo">1079</span> hRegion.replayWALFlushMarker(flushDesc, replaySeqId);<a name="line.1079"></a> -<span class="sourceLineNo">1080</span> continue;<a name="line.1080"></a> -<span class="sourceLineNo">1081</span> }<a name="line.1081"></a> -<span class="sourceLineNo">1082</span> RegionEventDescriptor regionEvent = WALEdit.getRegionEventDescriptor(metaCell);<a name="line.1082"></a> -<span class="sourceLineNo">1083</span> if (regionEvent != null && !isDefaultReplica) {<a name="line.1083"></a> -<span class="sourceLineNo">1084</span> hRegion.replayWALRegionEventMarker(regionEvent);<a name="line.1084"></a> -<span class="sourceLineNo">1085</span> continue;<a name="line.1085"></a> -<span class="sourceLineNo">1086</span> }<a name="line.1086"></a> -<span class="sourceLineNo">1087</span> BulkLoadDescriptor bulkLoadEvent = WALEdit.getBulkLoadDescriptor(metaCell);<a name="line.1087"></a> -<span class="sourceLineNo">1088</span> if (bulkLoadEvent != null) {<a name="line.1088"></a> -<span class="sourceLineNo">1089</span> hRegion.replayWALBulkLoadEventMarker(bulkLoadEvent);<a name="line.1089"></a> -<span class="sourceLineNo">1090</span> continue;<a name="line.1090"></a> -<span class="sourceLineNo">1091</span> }<a name="line.1091"></a> -<span class="sourceLineNo">1092</span> }<a name="line.1092"></a> -<span class="sourceLineNo">1093</span> it.remove();<a name="line.1093"></a> -<span class="sourceLineNo">1094</span> }<a name="line.1094"></a> -<span class="sourceLineNo">1095</span> }<a name="line.1095"></a> -<span class="sourceLineNo">1096</span> requestCount.increment();<a name="line.1096"></a> -<span class="sourceLineNo">1097</span> requestRowActionCount.add(mutations.size());<a name="line.1097"></a> -<span class="sourceLineNo">1098</span> if (!region.getRegionInfo().isMetaTable()) {<a name="line.1098"></a> -<span class="sourceLineNo">1099</span> regionServer.cacheFlusher.reclaimMemStoreMemory();<a name="line.1099"></a> -<span class="sourceLineNo">1100</span> }<a name="line.1100"></a> -<span class="sourceLineNo">1101</span> return region.batchReplay(mutations.toArray(<a name="line.1101"></a> -<span class="sourceLineNo">1102</span> new WALSplitter.MutationReplay[mutations.size()]), replaySeqId);<a name="line.1102"></a> -<span class="sourceLineNo">1103</span> } finally {<a name="line.1103"></a> -<span class="sourceLineNo">1104</span> if (regionServer.metricsRegionServer != null) {<a name="line.1104"></a> -<span class="sourceLineNo">1105</span> long after = EnvironmentEdgeManager.currentTime();<a name="line.1105"></a> -<span class="sourceLineNo">1106</span> if (batchContainsPuts) {<a name="line.1106"></a> -<span class="sourceLineNo">1107</span> regionServer.metricsRegionServer.updatePutBatch(after - before);<a name="line.1107"></a> -<span class="sourceLineNo">1108</span> }<a name="line.1108"></a> -<span class="sourceLineNo">1109</span> if (batchContainsDelete) {<a name="line.1109"></a> -<span class="sourceLineNo">1110</span> regionServer.metricsRegionServer.updateDeleteBatch(after - before);<a name="line.1110"></a> -<span class="sourceLineNo">1111</span> }<a name="line.1111"></a> -<span class="sourceLineNo">1112</span> }<a name="line.1112"></a> -<span class="sourceLineNo">1113</span> }<a name="line.1113"></a> -<span class="sourceLineNo">1114</span> }<a name="line.1114"></a> -<span class="sourceLineNo">1115</span><a name="line.1115"></a> -<span class
<TRUNCATED>