[GitHub] [ozone] symious commented on a diff in pull request #3350: HDDS-6648. Add Root in network-topology-default.xml

[GitBox <gi...@apache.org>]

symious commented on code in PR #3350:
URL: https://github.com/apache/ozone/pull/3350#discussion_r877899377


##########
hadoop-hdds/common/src/main/resources/network-topology-default.yaml:
##########
@@ -30,32 +30,26 @@ prefix: /
 type: ROOT
 
 # Layer name
-defaultName: root
+defaultName: datacenter
 
 # Sub layer
 # The sub layer property defines as a list which can reflect a node tree, though
 # in schema template it always has only one child.
 sublayer:
   -
     cost: 1
-    prefix: dc
-    defaultName: datacenter
+    prefix: rack
+    defaultName: rack
     type: INNER_NODE
     sublayer:
-      -
-        cost: 1
-        prefix: rack
-        defaultName: rack
-        type: INNER_NODE
-        sublayer:
+        -
+          cost: 1
+          prefix: ng
+          defaultName: nodegroup

Review Comment:
   @ChenSammi Thanks for the review. Updated the PR, please have a check.



-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org