swamirishi opened a new pull request, #3606: URL: https://github.com/apache/ozone/pull/3606
## What changes were proposed in this pull request? When the object of the Class DataNodeDetails is built from protobuf only UUID of the datanode is added which is used for the hashcode. Thus not passing any information about the topology. While excluding datanodes the object is built from protobuf. NetworkTopology removes all nodes from the list which does not fall under the scope while selecting a random node. Default scope value is "/default-rack/" which won't match the required scope. Thus pass the proper object of DatanodeDetails while trying to get the random node from NetworkTopology should fix this. ## What is the link to the Apache JIRA https://jira.cloudera.com/browse/CDPD-42577 ## How was this patch tested? Stage Test. Writing a unit test would be flaky. -- 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]
