adoroszlai commented on code in PR #4485: URL: https://github.com/apache/ozone/pull/4485#discussion_r1150203207
########## hadoop-hdds/docs/content/feature/Topology.md: ########## @@ -54,7 +54,7 @@ Static list can be configured with the help of ```TableMapping```: </property> ``` -The second configuration option should point to a text file. The file format is a two column text file, with columns separated by whitespace. The first column is a DNS or IP address and the second column specifies the rack where the address maps. If no entry corresponding to a host in the cluster is found, then `/default-rack` is assumed. +The second configuration option should point to a text file. The file format is a two column text file, with columns separated by whitespace. The first column is IP address and the second column specifies the rack where the address maps. If no entry corresponding to a host in the cluster is found, then `/default-rack` is assumed. Review Comment: This text, including support for DNS addresses, is adapted from `TableMapping` javadoc. https://github.com/apache/hadoop/blob/a585a73c3e02ac62350c136643a5e7f6095a3dbb/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/net/TableMapping.java#L42-L53 Where is the limitation of "only IP address is supported" is coming from? -- 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]
