siddhantsangwan commented on pull request #2786:
URL: https://github.com/apache/ozone/pull/2786#issuecomment-963028954


   > thanks @siddhantsangwan for this work, the changes generally looks good! i 
have a concern that if we have many Include Nodes and Exclude Nodes, how can we 
pass these host names or IP addresses to balancer? typing these host names or 
IP addresses one by one in command line , or specifying a local file in command 
line which contains these informations.
   
   We can have setters in `ContainerBalancerConfiguration` that accept a file 
containing hostnames. At the command line, can we have it such that either a 
string of hostnames or a file can be specified?


-- 
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]

Reply via email to