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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]