You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by sk...@apache.org on 2020/05/12 01:30:43 UTC

[phoenix] branch master updated: PHOENIX-5870: IndexRegionObserver should retry before mappers in case of rebuild (#770)

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

skadam pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/phoenix.git


The following commit(s) were added to refs/heads/master by this push:
     new 9431b84  PHOENIX-5870: IndexRegionObserver should retry before mappers in case of rebuild (#770)
9431b84 is described below

commit 9431b849a486d3da1e7f8c063fa64b044559fa82
Author: Swaroopa Kadam <sw...@gmail.com>
AuthorDate: Mon May 11 18:07:52 2020 -0700

    PHOENIX-5870: IndexRegionObserver should retry before mappers in case of rebuild (#770)
    
    Co-authored-by: s.kadam <s....@apache.org>
---
 .../java/org/apache/phoenix/hbase/index/IndexRegionObserver.java    | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/phoenix-core/src/main/java/org/apache/phoenix/hbase/index/IndexRegionObserver.java b/phoenix-core/src/main/java/org/apache/phoenix/hbase/index/IndexRegionObserver.java
index 1b13153..a76ba51 100644
--- a/phoenix-core/src/main/java/org/apache/phoenix/hbase/index/IndexRegionObserver.java
+++ b/phoenix-core/src/main/java/org/apache/phoenix/hbase/index/IndexRegionObserver.java
@@ -1067,7 +1067,11 @@ public class IndexRegionObserver implements RegionObserver, RegionCoprocessor {
           // postBatchMutateIndispensably() is called
           removePendingRows(context);
           context.rowLocks.clear();
-          rethrowIndexingException(e);
+          if (context.rebuild) {
+              throw new IOException(String.format("%s for rebuild", e.getMessage()));
+          } else {
+              rethrowIndexingException(e);
+          }
       }
       throw new RuntimeException(
               "Somehow didn't complete the index update, but didn't return succesfully either!");