[ 
https://issues.apache.org/jira/browse/YARN-7191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16181132#comment-16181132
 ] 

Billie Rinaldi commented on YARN-7191:
--------------------------------------

Thanks for the documentation patch, [~jianhe]! Some comments:
* Instead of using "http://myContainer.myDomain.com/endpoint"; as an example 
endpoint, I think it would be better to use an example URL in the actual 
registry DNS format. We could reuse the hbase master endpoint example, or 
another example such as 
"http://solr-0.solr-service.devuser.yarncluster:8983/solr/admin/collections?action=LIST";
* /28 16 IPs seems like a small number. I think we should change the example to 
use /27 (32 IPs) or /26 (64 IPs) so we don't give the wrong impression that 
only a few IPs can be assigned for docker containers.
* Instead of dev, we should name the example user something like devuser so 
that it is clear from the URL which element is the user.

It's a little confusing that this patch contains some of my documentation 
changes from YARN-7198 but not all of them. Do you think we should make all the 
doc changes in this patch and remove them from the other patch, or should I 
plan to do the remaining doc cleanup in YARN-7198?

> Improve yarn-service documentation
> ----------------------------------
>
>                 Key: YARN-7191
>                 URL: https://issues.apache.org/jira/browse/YARN-7191
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Jian He
>            Assignee: Jian He
>         Attachments: YARN-7191.yarn-native-services.01.patch, 
> YARN-7191.yarn-native-services.02.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to