ChenSammi commented on a change in pull request #931: HDDS-1586. Allow Ozone RPC client to read with topology awareness. URL: https://github.com/apache/hadoop/pull/931#discussion_r302015013
########## File path: hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OzoneManager.java ########## @@ -2347,7 +2347,7 @@ public OmKeyInfo lookupKey(OmKeyArgs args) throws IOException { boolean auditSuccess = true; try { metrics.incNumKeyLookups(); - return keyManager.lookupKey(args); + return keyManager.lookupKey(args, getClientAddress()); Review comment: No, use Ip address or hostname depends on the value of "dfs.datanode.use.datanode.hostname" property. If it's true, then use hostname, if it's false, the use Ipaddress. The default value of this property is false. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org