shahrs87 commented on code in PR #2198:
URL: https://github.com/apache/phoenix/pull/2198#discussion_r2162363187


##########
phoenix-core-server/src/main/java/org/apache/phoenix/hbase/index/IndexRegionObserver.java:
##########
@@ -540,8 +541,9 @@ public void 
preBatchMutate(ObserverContext<RegionCoprocessorEnvironment> c,
           final Configuration conf = c.getEnvironment().getConfiguration();
           final HAGroupStoreManager haGroupStoreManager = 
HAGroupStoreManager.getInstance(conf);
           if (haGroupStoreManager.isMutationBlocked()) {
-              throw new IOException("Blocking Mutation as Some CRRs are in 
ACTIVE_TO_STANDBY "
-                      + "state and CLUSTER_ROLE_BASED_MUTATION_BLOCK_ENABLED 
is true");
+               throw new MutationBlockedIOException("Blocking Mutation as Some 
CRRs "

Review Comment:
   > There can be multiple CRRs
   
   For a given cluster on the server side, it will always watch it owns local 
ZK's CRR. Even if this cluster is marked ATS on some other cluster's CRR, how 
will this cluster get notified when we change the role in other cluster's CRR? 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@phoenix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to