[
https://issues.apache.org/jira/browse/YARN-5740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15753103#comment-15753103
]
Jian He commented on YARN-5740:
-------------------------------
thanks for the review, fixed the issues except one, which I think no need to
fix because all other variables do the same.
{{./hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-slider/hadoop-yarn-slider-core/src/main/java/org/apache/slider/common/params/ActionStatusArgs.java:40:
public boolean lifetime;:18: Variable 'lifetime' must be private and have
accessor methods.}}
> Add a new field in Slider status output - lifetime (remaining)
> --------------------------------------------------------------
>
> Key: YARN-5740
> URL: https://issues.apache.org/jira/browse/YARN-5740
> Project: Hadoop YARN
> Issue Type: Sub-task
> Reporter: Gour Saha
> Assignee: Jian He
> Fix For: yarn-native-services
>
> Attachments: YARN-5740-yarn-native-services.01.patch,
> YARN-5740-yarn-native-services.02.patch
>
>
> With YARN-5735, REST service is now setting lifetime to application during
> submission (YARN-4205 exposed application lifetime support). Now Slider
> status needs to expose this field so that the REST service can return it in
> its GET response. Note, the lifetime value that GET response intends to
> return is the remaining lifetime of the application.
> There is an ongoing discussion in YARN-4206, that the lifetime value returned
> in Application Report will be the remaining lifetime (at the time of
> request). So until it is finalized, the lifetime value might go through
> different connotations. But as long as we have the lifetime field in the
> status output, it will be a good start.
--
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
