echonesis commented on code in PR #10591:
URL: https://github.com/apache/ozone/pull/10591#discussion_r3489600143


##########
hadoop-ozone/common/src/test/java/org/apache/hadoop/ozone/om/protocolPB/TestS3GrpcOmTransport.java:
##########
@@ -272,6 +282,214 @@ public void testGrpcFailoverExceedMaxMesgLen() throws 
Exception {
     assertThrows(Exception.class, () -> client.submitRequest(omRequest));
   }
 
+  @Test
+  public void testFollowerReadSkipsKnownLeader() throws Exception {

Review Comment:
   I changed the test to explicitly set the follower-read initial proxy to the 
known leader. 
   The test now verifies that gRPC follower read sends the request to that OM 
directly and does not fail over to the other OM.



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to