hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Thomas Graves (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-1407) apps REST API filters queries by YarnApplicationState, but includes RMAppStates in response
Date Wed, 13 Nov 2013 15:25:56 GMT

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

Thomas Graves commented on YARN-1407:
-------------------------------------

 I think it will be fine for backwards compatibility since as you said we are reducing what
it could return, so it atleast won't be any new unexpected values.   We should update the
documentation:  http://hadoop.apache.org/docs/r2.2.0/hadoop-yarn/hadoop-yarn-site/ResourceManagerRest.html#Cluster_Application_API.
It might be best just to remove the hardcoded valid values and point to the YarnApplicationState
for valid states. 

> apps REST API filters queries by YarnApplicationState, but includes RMAppStates in response
> -------------------------------------------------------------------------------------------
>
>                 Key: YARN-1407
>                 URL: https://issues.apache.org/jira/browse/YARN-1407
>             Project: Hadoop YARN
>          Issue Type: Bug
>    Affects Versions: 2.2.0
>            Reporter: Sandy Ryza
>            Assignee: Sandy Ryza
>         Attachments: YARN-1407.patch
>
>
> RMAppState isn't a public facing enum like YarnApplicationState, so we shouldn't return
values that come from it.
> It is not 100% clear to me whether or not fixing this would be a backwards-incompatible
change.  The change would only reduce the set of possible strings that the API returns, so
I think not.  We have also been changing the contents of RMAppState since 2.2.0, e.g. in YARN-891.
It would still be good to fix this ASAP (i.e. for 2.2.1).



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Mime
View raw message