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

Wangda Tan commented on YARN-3866:
----------------------------------

Thanks [~ajisakaa], [~djp] for working on 2.8 release and check JACC report.

This problem is actually different from typical incompatibility change.

APIs which are modified are originally added by YARN-1447, and no 
implementations at the back end. Which means:
- The removed fields are never used by any of YARN modules before 2.8.
- And of course, we have never declares it works before 2.8.

We discussed this with [~jianhe], [~mding] and [~asuresh] offline. Since nobody 
suppose to use this field unless by mistake, to make a cleaner interface, we 
made decision to change the field directly.

And in addition, the AMRMProtocol is lower level API of YARN, according to my 
knowledge, the only few project AMRMProtocol which I knew is MR. YARN 
application is supposed to use AMRMClient APIs.

So to me, the incompatible change should have minimum impact to downstream 
project. I'm OK with adding these fields back if everybody still agrees, but I 
want to get more input before proceed.

> 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