hanishakoneru commented on pull request #2765: URL: https://github.com/apache/ozone/pull/2765#issuecomment-952091699
@JacksonYao287, previously the client would failover to the suggested leader. It was changed in HDDS-3334 as it caused issues if the client configs are not in line with the server configs. For example, let's say the server nodeId to OM address mapping is as follows: ``` om1 -> node1 om2 -> node2 om3 -> node3 ``` But the client configuration has this mapping as follows: ``` om1 -> node2 om2 -> node3 om1 -> node1 ``` Let's say the leader is node1. Client contacts node2 and gets back suggested leader as om1 (as per the server). So it fails over to om1 (which according to its configuration is node2). So it contacts node2 again and this loop is repeated and client never fails over to the correct leader. Though generally we don't expect client and server mappings to be different, it is possible to have. So, it would be good to failover based on node address than to rely just on the nodeId. -- 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]
