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

Junping Du commented on YARN-3866:
----------------------------------

bq.  As Wangda Tan mentioned, the ApplicationMasterProtocol is a lower level 
protocol and end user applications should be using the AMRMClient.
I agree most applications are supposed to use AMRMClient (except MR). However, 
ApplicationMasterProtocol is still marked as public for using by downstream 
projects (open or closed sourced). Unless this API is marked as restricted to 
MR only, there should be 0 assumption for users on how to use it.

bq. Given that the increase/decrease resources are not available in any current 
version of hadoop.and given that YARN-5221 was raised to unify all container 
updates (resources and ExecutionType etc.) into a single API, I am not in favor 
of adding back the (set/get)(Increase/Decrease)(Requests/Containers) API as it 
will cause problems for future upgrades, if people do start using them.
No. It is already get released in 2.7.3: 
https://hadoop.apache.org/docs/r2.7.3/api/index.html. In release prospective, 
all released public APIs (unless marked as unstable explicitly) are protocol 
between software (and developer behind it) and users. I can understand that 
public these APIs in 2.7.3 may not be intentional, but we should take response 
for all sequences it could generate. We can deprecate these APIs to tell people 
to use the right API. I think that is the right way to do.

> AM-RM protocol changes to support container resizing
> ----------------------------------------------------
>
>                 Key: YARN-3866
>                 URL: https://issues.apache.org/jira/browse/YARN-3866
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: api
>            Reporter: MENG DING
>            Assignee: MENG DING
>             Fix For: 2.8.0, 3.0.0-alpha1
>
>         Attachments: YARN-3866-YARN-1197.4.patch, YARN-3866.1.patch, 
> YARN-3866.2.patch, YARN-3866.3.patch
>
>
> YARN-1447 and YARN-1448 are outdated. 
> This ticket deals with AM-RM Protocol changes to support container resize 
> according to the latest design in YARN-1197.
> 1) Add increase/decrease requests in AllocateRequest
> 2) Get approved increase/decrease requests from RM in AllocateResponse
> 3) Add relevant test cases



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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