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

ASF GitHub Bot commented on YARN-11524:
---------------------------------------

slfan1989 commented on code in PR #5797:
URL: https://github.com/apache/hadoop/pull/5797#discussion_r1258776847


##########
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/Federation.md:
##########
@@ -349,6 +430,10 @@ Optional:
 | `yarn.federation.statestore.max-connections` | `1` | The maximum number of 
parallel connections from each AMRMProxy to the state-store. This value is 
typically lower than the router one, since we have many AMRMProxy that could 
burn-through many DB connections quickly. |
 | `yarn.federation.cache-ttl.secs` | `300` | The time to leave for the 
AMRMProxy cache. Typically larger than at the router, as the number of 
AMRMProxy is large, and we want to limit the load to the centralized 
state-store. |
 
+### ON GPG:

Review Comment:
   Thank you very much for your help in reviewing the code! `GPG` will submit 
pr separately to improve the description of md.





> Improve the Policy Description in Federation.md
> -----------------------------------------------
>
>                 Key: YARN-11524
>                 URL: https://issues.apache.org/jira/browse/YARN-11524
>             Project: Hadoop YARN
>          Issue Type: Improvement
>          Components: federation
>    Affects Versions: 3.4.0
>            Reporter: Shilun Fan
>            Assignee: Shilun Fan
>            Priority: Major
>              Labels: pull-request-available
>
> The Federation.md file lacks an introduction to Policy. In this Jira ticket, 
> we will enhance the content regarding Router Policy, AMRM Policy, and 
> PolicyManager to help users gain a better understanding of these types of 
> policies.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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

Reply via email to