[GitHub] eagle issue #787: [MINOR] Change default-rack to /default-rack

2017-01-19 Thread r7raul1984
Github user r7raul1984 commented on the issue: https://github.com/apache/eagle/pull/787 @qingwen220 Because ClusterNodeAPITopologyRackResolver use "/default-rack" .I think it will issue some problem when you complete EAGLE-876. --- If your project is set up for it, you can reply

[GitHub] eagle issue #787: [MINOR] Change default-rack to /default-rack

2017-01-19 Thread qingwen220
Github user qingwen220 commented on the issue: https://github.com/apache/eagle/pull/787 At the initial design, we prefer to extract the 'exact rack info', for example, '/rack-45' -> 'rack-45', '/' will be removed. --- If your project is set up for it, you can reply to this email