You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by en...@apache.org on 2013/10/09 19:42:29 UTC
svn commit: r1530729 -
/hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
Author: enis
Date: Wed Oct 9 17:42:29 2013
New Revision: 1530729
URL: http://svn.apache.org/r1530729
Log:
HBASE-9730 Exceptions in multi operations are not handled correctly
Modified:
hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
Modified: hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java?rev=1530729&r1=1530728&r2=1530729&view=diff
==============================================================================
--- hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java (original)
+++ hbase/branches/0.96/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java Wed Oct 9 17:42:29 2013
@@ -4020,8 +4020,6 @@ public class HRegionServer implements Cl
try {
int i = 0;
for (ClientProtos.Action action: mutations) {
- ClientProtos.ResultOrException.Builder resultOrExceptionBuilder =
- ClientProtos.ResultOrException.newBuilder();
MutationProto m = action.getMutation();
Mutation mutation;
if (m.getMutateType() == MutationType.PUT) {
@@ -4065,9 +4063,8 @@ public class HRegionServer implements Cl
}
}
} catch (IOException ie) {
- ResultOrException resultOrException = ResponseConverter.buildActionResult(ie).build();
for (int i = 0; i < mutations.size(); i++) {
- builder.addResultOrException(resultOrException);
+ builder.addResultOrException(getResultOrException(ie, mutations.get(i).getIndex()));
}
}
long after = EnvironmentEdgeManager.currentTimeMillis();