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

Daniel Templeton commented on YARN-6742:
----------------------------------------

Thanks for the patch.  Overall a great set of language edits.  Here are my 
additional fixes:

* "merely meaning that the process are running" => "merely meaning that the 
processes are running"
* "register a web URL and an IPC address. But it is not sufficient for our 
purposes since it does not allow" => "register a web URL and an IPC address. It 
is not sufficient for our purposes, however, as it does not allow"
* "to be registered —such as" => "to be registered—such as"
* "able to register certificates and other public security information" => 
"able to register certificates, and other public security information"
* "There has no attempt to restrict read access" => "There will be no attempt 
to restrict read access"
* "security restriction (this is why" => "security restriction. (This is why"
* "all other accounts, and anonymous access is read-only." => "all other 
accounts and anonymous access are read-only."
* {quote}* The permissions are similarly restricted for `/users`, and 
`/services/`

* installations may extend or change these system accounts.{quote} => {quote}* 
The permissions are similarly restricted for `/users`. `/services/`
installations may extend or change these system accounts.{quote}
* "add extra `digest:` ACL tokens" => "add extra `digest`: ACL tokens,"  <-- 
Note the trailing comma.
* "The agents use two-way HTTPS authentication-—if the AM fails" => "The agents 
use two-way HTTPS authentication. If the AM fails"

> Minor mistakes in "The YARN Service Registry" docs
> --------------------------------------------------
>
>                 Key: YARN-6742
>                 URL: https://issues.apache.org/jira/browse/YARN-6742
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: documentation
>    Affects Versions: 3.0.0-alpha3
>            Reporter: Yeliang Cang
>            Assignee: Yeliang Cang
>            Priority: Trivial
>         Attachments: YARN-6742-001.patch, YARN-6742-002.patch, 
> YARN-6742-003.patch
>
>
> There are minor mistakes in The YARN Service Registry docs.



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to