huaxiangsun commented on code in PR #4442:
URL: https://github.com/apache/hbase/pull/4442#discussion_r877404672


##########
hbase-server/src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestMetaRegionReplicaReplication.java:
##########
@@ -477,7 +495,7 @@ public void testHBaseMetaReplicaGets() throws Exception {
 
       // There are read requests increase for primary meta replica.
       // For rest of meta replicas, there is no change as regionMove will tell 
the new location
-      primaryIncreaseReplicaNoChange(readReqsForMetaReplicasAfterGet,
+      
primaryIncreaseReplicaNoChange(readReqsForMetaReplicasAfterGetAllLocations,

Review Comment:
   All locations are cached at the client side, when a region move, for a short 
period of time, the previous RS maintains a moved region list. Get requests to 
previous RS result in RegionMovedException, which has the new RS info. There is 
no new requests to meta replica regions in this case.



-- 
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...@hbase.apache.org

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

Reply via email to