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