You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by vj...@apache.org on 2024/03/10 08:44:33 UTC

(hbase) branch branch-3 updated: HBASE-28424 Set correct Result to RegionActionResult for successful Put/Delete mutations (#5760)

This is an automated email from the ASF dual-hosted git repository.

vjasani pushed a commit to branch branch-3
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-3 by this push:
     new fe6c4cb2417 HBASE-28424 Set correct Result to RegionActionResult for successful Put/Delete mutations (#5760)
fe6c4cb2417 is described below

commit fe6c4cb24170eb92367fcb7693ce2192eb2e389f
Author: Jing Yu <yu...@salesforce.com>
AuthorDate: Sun Mar 10 00:43:46 2024 -0800

    HBASE-28424 Set correct Result to RegionActionResult for successful Put/Delete mutations (#5760)
    
    Signed-off-by: Viraj Jasani <vj...@apache.org>
---
 .../java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java    | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
index 25b229fd0c4..7043b78c048 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
@@ -1046,8 +1046,10 @@ public class RSRpcServices extends HBaseRpcServicesBase<HRegionServer>
             break;
 
           case SUCCESS:
-            builder.addResultOrException(
-              getResultOrException(ClientProtos.Result.getDefaultInstance(), index));
+            ClientProtos.Result result = codes[i].getResult() == null
+              ? ClientProtos.Result.getDefaultInstance()
+              : ProtobufUtil.toResult(codes[i].getResult());
+            builder.addResultOrException(getResultOrException(result, index));
             break;
 
           case STORE_TOO_BUSY: